Messages in this thread | | | From | Nick Desaulniers <> | Date | Mon, 11 May 2020 11:46:09 -0700 | Subject | Re: [PATCH] x86: support i386 with Clang |
| |
On Mon, May 11, 2020 at 11:09 AM Brian Gerst <brgerst@gmail.com> 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
| |