[clang] [Clang][Sema] Push an evaluation context for type constraints (PR #93945)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Fri May 31 06:36:48 PDT 2024


================
@@ -5004,6 +5004,20 @@ static bool CheckDeducedPlaceholderConstraints(Sema &S, const AutoType &Type,
     return true;
   MultiLevelTemplateArgumentList MLTAL(Concept, CanonicalConverted,
                                        /*Final=*/false);
+  // Build up an EvaluationContext with an ImplicitConceptSpecializationDecl so
+  // that the template arguments of the constraint can be preserved. For
+  // example:
+  //
+  //  template <class T>
+  //  concept C = []<D U = void>() { return true; }();
+  //
+  // We need the argument for T while evaluating type constraint D in
+  // building the CallExpr to the lambda.
+  EnterExpressionEvaluationContext EECtx(
+      S, Sema::ExpressionEvaluationContext::ConstantEvaluated,
----------------
erichkeane wrote:

I think Corentin is right.  And that link from SemaTemplate might actually be wrong too, but I'm not completely sure.  Though I would have expected us to see a bunch of bugs for evaluating that incorreclty.

https://github.com/llvm/llvm-project/pull/93945


More information about the cfe-commits mailing list