cor3ntin added inline comments.
================ Comment at: clang/lib/Sema/SemaLambda.cpp:1378-1383 + CompleteLambdaCallOperator( + Method, Intro.Range.getBegin(), ParamInfo.getTrailingRequiresClause(), + MethodTyInfo, ParamInfo.getDeclSpec().getConstexprSpecifier(), Params, + ExplicitResultType); + ContextRAII ManglingContext(*this, Class->getDeclContext()); ---------------- aaron.ballman wrote: > The previous logic was to enter a mangling context and then check params, but > the new logic checks params outside of the mangling context. Is that > intentional? It has no impact one way or another, but we can't call `CompleteLambdaCallOperator` in the context of the class, so this is simpler. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124012/new/ https://reviews.llvm.org/D124012 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
