[PATCH] sizeof a packexpansion should be in an unevaluated context

[Faisal Vali]

Hi rsmith, doug.gregor,


    [](auto ... a) {
       return [](auto b) {
          return sizeof...(a); <-- this should be in an unevaluated context
       }
   }


http://llvm-reviews.chandlerc.com/D2029

Files:
  ParseExpr.cpp

Index: ParseExpr.cpp
===================================================================
--- ParseExpr.cpp
+++ ParseExpr.cpp
@@ -1711,6 +1711,9 @@
     if (!Name)
       return ExprError();
     
+    EnterExpressionEvaluationContext Unevaluated(Actions, Sema::Unevaluated,
+                                                 Sema::ReuseLambdaContextDecl);
+
     return Actions.ActOnSizeofParameterPackExpr(getCurScope(),
                                                 OpTok.getLocation(), 
                                                 *Name, NameLoc,

Index: ParseExpr.cpp
===================================================================
--- ParseExpr.cpp
+++ ParseExpr.cpp
@@ -1711,6 +1711,9 @@
     if (!Name)
       return ExprError();
     
+    EnterExpressionEvaluationContext Unevaluated(Actions, Sema::Unevaluated,
+                                                 Sema::ReuseLambdaContextDecl);
+
     return Actions.ActOnSizeofParameterPackExpr(getCurScope(),
                                                 OpTok.getLocation(), 
                                                 *Name, NameLoc,

_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits