cor3ntin added a subscriber: erichkeane.
cor3ntin added a comment.
This looks like the right fix to me but I would prefer @aaron.ballman or
@erichkeane to have the final say.
================
Comment at: clang/lib/Sema/SemaExpr.cpp:16731
/*IsImmediateInvocation*/ true);
- ExprEvalContexts.back().ImmediateInvocationCandidates.emplace_back(Res, 0);
+ /// Value-dependent constant expression are not meant to be calculated.
+ /// Each template instantiation will calculate its own value later.
----------------
I'd say something like
/// Value-dependent constant expressionS should not be immediately
// evaluated until they are instantiated.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119375/new/
https://reviews.llvm.org/D119375
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
