[PATCH] D132659: PotentiallyEvaluatedContext in a ImmediateFunctionContext.

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added reviewers: cor3ntin, aaron.ballman, clang-language-wg.
aaron.ballman added a comment.

Adding a few more reviewers just in case, but this is generally looking good. Please also add a release note for the fix!



================
Comment at: clang/include/clang/Sema/Sema.h:1355
+              InImmediateFunctionContext) ||
+             // [expr.const#def:immediate_function_context]
+             // An expression or conversion is in an immediate function
----------------
cor3ntin wrote:
> 



================
Comment at: clang/include/clang/Sema/Sema.h:1357-1359
+             // context if it is potentially evaluated and either: its
+             // innermost enclosing non-block scope is a function parameter
+             // scope of an immediate function.
----------------



================
Comment at: clang/lib/Sema/SemaDecl.cpp:14774
   if (!isLambdaCallOperator(FD))
+    // [expr.const#def:immediate_function_context]
+    // An expression or conversion is in an immediate function context if it is
----------------
cor3ntin wrote:
> 



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D132659/new/

https://reviews.llvm.org/D132659