erik.pilkington added a comment. > LGTM - can you commit?
Yep, I'll do that now. Thanks for reviewing! ================ Comment at: clang/lib/AST/ExprConstant.cpp:5212 + cast<DeclRefExpr>(E)->refersToEnclosingVariableOrCapture()) { + // We don't track the lambda's captures in a potential constant expression. + if (Info.checkingPotentialConstantExpression()) ---------------- faisalv wrote: > How about something along the lines of : We don't always have a complete > capture-map when checking or inferring if the function call operator meets > the requirements of a constexpr function - and we don't need to evaluate the > captures to determine constexprness (dcl.constexpr C++17)? Sure, I just copied that verbatim into the commit. https://reviews.llvm.org/D45194 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits