[PATCH] D120398: [format] follow up: Use unsigned char as the base of all enums in FormatStyle

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 27 04:53:12 PST 2022


curdeius added a comment.

Looks innocent but I have a few questions.
What's the size difference that you observe?
Any runtime penalty? (Usually compilers choose a faster underlying type for enums, not smaller)
Also, is this change very relevant to clang-format? We should not copy FormatStyle a lot, no?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120398



More information about the cfe-commits mailing list