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