[PATCH] D135088: [Clang] make canonical AutoType constraints-free

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 3 10:24:33 PDT 2022


erichkeane accepted this revision.
erichkeane added a comment.
This revision is now accepted and ready to land.

This looks right enough to me with the 1 nit, but let Matheus take a look before committing.



================
Comment at: clang/lib/AST/ASTContext.cpp:699
       ID.AddBoolean(NTTP->isParameterPack());
+      const Expr *Exp = NTTP->getPlaceholderTypeConstraint();
+      ID.AddBoolean(Exp != nullptr);
----------------
As a nit, I'd probably just call this `TC` or `TypeConstr`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135088



More information about the cfe-commits mailing list