[PATCH] Tame a few enum size tests when using -fshort-enums on ARM.
Eric Fiselier
eric at efcs.ca
Fri Aug 15 11:25:21 PDT 2014
> I'm not sure I understand... This works on ARM for both -fshort-enums and -fno-short-enums.
Sorry, I meant the original test fails whenever -fshort-enums is used. Not just on ARM. Your patch is definitely a lot better at calling out the weird behavior. Optimally I would like to make the workaround less ARM specific, like how wchar_t is handled.
I'm not convinced that we are going to run into cases where `sizeof(Enum)` is not the same as `sizeof(int)` or `sizeof(char)`.
I apologize. I'm not trying to be difficult. My main concern is introducing a workaround that only a very small set of users can test and reason about. If '-fshort-enums' is causing this failure then I want to work around the switch and not how an architecture handles the switch.
http://reviews.llvm.org/D4869
More information about the cfe-commits
mailing list