================ @@ -34,9 +34,15 @@ const char *getTokenTypeName(TokenType Type) { return nullptr; } +// Sorted common C++ non-keyword types. +static SmallVector<StringRef> CppNonKeywordTypes = { + "byte", "int16_t", "int32_t", "int64_t", "int8_t", ---------------- zygoloid wrote:
Does `byte` really belong here? It's not available as an unqualified name, only as `std::byte`. I could see an argument for including `nullptr_t` here because it is available unqualified, but even that seems a little hard to justify since I don't think people write that often. I also expect people use `byte` as a variable name frequently; the same doesn't seem to be true for these other names. https://github.com/llvm/llvm-project/pull/83709 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits