[PATCH] D120398: [format] follow up: Use unsigned char as the base of all enums in FormatStyle
MyDeveloperDay via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Mar 1 10:05:09 PST 2022
MyDeveloperDay added a comment.
Before this lands can we have a discussion about what clarity this gives us?, because I think it makes the code more unreadable, but surely we are saving just 7x(3 bytes) (the difference between and int and a unsigned char for 7 enums)
Is saving 21 bytes valuable?
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