================
@@ -1028,7 +1021,7 @@ ExprResult ConstraintSatisfactionChecker::EvaluateSlow(
       ConceptId->getFoundDecl(), ConceptId->getNamedConcept(), &OutArgs,
       /*DoCheckConstraintSatisfaction=*/false);
 
-  if (SubstitutedConceptId.isInvalid() || Trap.hasErrorOccurred())
+  if (SubstitutedConceptId.isInvalid())
     return ExprError();
 
----------------
cor3ntin wrote:

> FWIW we should get rid of uses of hasErrorOccurred(), in favor of not 
> performing error recovery in SFINAE context,

Maybe - but either way, why is this change necessary in the context of this PR?

https://github.com/llvm/llvm-project/pull/207966
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to