rsmith added inline comments.

================
Comment at: include/clang/Parse/Parser.h:1432
+  ExprResult ParseConstantExpression(TypeCastState isTypeCast = NotTypeCast,
+                                     bool IsLambdaExprForbidden = false);
   ExprResult ParseConstraintExpression();
----------------
I would call this `IsInSignature`.


================
Comment at: include/clang/Sema/Sema.h:894-895
 
+    /// \brief Whether lambda expressions are forbidden here.
+    bool IsLambdaExprForbidden;
+
----------------
Rather than adding a flag, how about we have two different kinds of 
`ExpressionEvaluationContext` for constant expressions: a generic "constant 
expression" context and a "constant expression in signature" context?


Repository:
  rL LLVM

https://reviews.llvm.org/D28510



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to