================
@@ -1131,11 +1153,11 @@ ExprResult ConstraintSatisfactionChecker::Evaluate(
   // ConceptIdConstraint is only relevant for diagnostics,
   // so if the normalized constraint is satisfied, we should not
   // substitute into the constraint.
-  if (Satisfaction.IsSatisfied)
+  if (Satisfaction.IsSatisfied) {
+    if (!BuildExpression && !Satisfaction.ContainsErrors)
+      PMCache.cacheSatisfied();
     return E;
-
-  ParameterMappingInstantiationCache PMCache(
-      *this, Constraint, MLTAL, getOuterPackIndex(Constraint), Size);
+  }
----------------
shafik wrote:

This seems worth adding a comment for.

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

Reply via email to