================
@@ -18931,7 +18931,7 @@ void Sema::MarkFunctionReferenced(SourceLocation Loc,
FunctionDecl *Func,
// constant evaluated
bool NeededForConstantEvaluation =
isPotentiallyConstantEvaluatedContext(*this) &&
- isImplicitlyDefinableConstexprFunction(Func);
+ isImplicitlyDefinableConstexprFunction(Func) && !Func->isPure();
----------------
charmitro wrote:
Moving it to `NeedDefinition` silences the warnings of test
`call_pure_virtual_function_from_virtual`, where we still require warnings for
it, as mentioned in
https://github.com/llvm/llvm-project/issues/74016#issuecomment-1845636487.
https://github.com/llvm/llvm-project/pull/74510
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
