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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to