Reject lambdas in Build instead of ActOn

Author: Sirraide

clang/lib/Sema/SemaExpand.cpp

@@ -253,13 +253,6 @@ StmtResult Sema::ActOnCXXExpansionStmtPattern(
   if (DiagnoseUnexpandedParameterPack(ExpansionInitializer))
     return StmtError();
 
-  // Reject lambdas early.
-  if (auto *RD = ExpansionInitializer->getType()->getAsCXXRecordDecl();
-      RD && RD->isLambda()) {
-    Diag(ExpansionInitializer->getBeginLoc(), diag::err_expansion_stmt_lambda);
-    return StmtError();
-  }
-
   return BuildNonEnumeratingCXXExpansionStmtPattern(
       ESD, Init, DS, ExpansionInitializer, LParenLoc, ColonLoc, RParenLoc,
       LifetimeExtendTemps);
@@ -298,6 +291,13 @@ StmtResult Sema::BuildNonEnumeratingCXXExpansionStmtPattern(
     return StmtError();
   }
 
+  // Reject lambdas early.
+  if (auto *RD = ExpansionInitializer->getType()->getAsCXXRecordDecl();
+      RD && RD->isLambda()) {
+    Diag(ExpansionInitializer->getBeginLoc(), diag::err_expansion_stmt_lambda);
+    return StmtError();
+  }
+
   // Otherwise, if it can be an iterating expansion statement, it is one.
   DeclRefExpr *Index = BuildIndexDRE(*this, ESD);
   IterableExpansionStmtData Data = TryBuildIterableExpansionStmtInitializer(