aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
Thanks, this generally looks good but I did spot a small change to make (can make it when landing). ================ Comment at: clang/include/clang/AST/Type.h:1542 +static const unsigned NumOfBuiltinTypeBits = 8; + ---------------- I think this should move down to be directly above the definition of `BuiltinTypeBitfields`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D152429/new/ https://reviews.llvm.org/D152429 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
