On Mon, May 11, 2020 at 12:52 PM Nick Desaulniers <ndesaulni...@google.com> wrote: > > Interesting approach. Researching __builtin_choose_expr, it looks > like it was cited as prior art for C11's _Generic keyword.
Well, the thing that made me think that __builtin_choose_expr() would work is that unlike the switch statement, you absolutely _have_ to do the choice in the front end. You can't leave it as some kind of optimization for later phases, because the choice od expression ends up also determining the type of the result, so it isn't just a local choice - it affects everything around that expression. But clang still doesn't like that "qi" constraint with a (non-chosen) expression that has a "u64" type. I guess we can take the stupid extra cast, but I think it would at least need a comment (maybe through a helper function) about why "qi" needs it, but "ri" does not, and why the cast to "unsigned long" is needed, even though "clearly" the type is already just 8 bits. Otherwise somebody will just remove that "obviously pointless" cast, and gcc will eat the result happily, and clang will fail. And nobody will notice for a while anyway, because this issue only happens on 32-bit targets, and developers don't use those any more. Linus