[PATCH] D124923: [Sema] Simplify CheckConstraintSatisfaction. NFC

Ilya Biryukov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 4 07:01:36 PDT 2022


ilya-biryukov added inline comments.


================
Comment at: clang/lib/Sema/SemaConcept.cpp:335
+  auto Satisfaction =
+      std::make_unique<ConstraintSatisfaction>(Template, TemplateArgs);
   if (::CheckConstraintSatisfaction(*this, Template, ConstraintExprs,
----------------
I also wonder if this could be allocated in the AST arena, maybe worth exploring.
Definitely outside this change, though, would like to keep this one an NFC.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124923/new/

https://reviews.llvm.org/D124923



More information about the cfe-commits mailing list