Re: [PATCH] x86: support i386 with Clang

From: Nick Desaulniers
Date: Mon May 11 2020 - 14:46:23 EST


On Mon, May 11, 2020 at 11:09 AM Brian Gerst <brgerst@xxxxxxxxx> wrote:
> This looks like the same issue that we just discussed for bitops.h.
> Add the "b" operand size modifier to force it to use the 8-bit
> register names (and probably also needs the "w" modifier in the 16-bit
> case).

While it does feel familiar, it is slightly different.
https://godbolt.org/z/Rme4Zg
That case was both compilers validating the inline asm, yet generating
assembly that the assembler would choke on. This case is validation
in the front end failing.

Side note: would you mind sending a review by tag for v5 of that patch
if you think it's good to go? It does fix a regression I'd prefer
didn't ship in 5.7.
--
Thanks,
~Nick Desaulniers