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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
