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

[MyDeveloperDay via Phabricator via cfe-commits]

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