[ expanded Cc ] Linus Torvalds <torva...@linux-foundation.org> writes: > On Sat, Aug 10, 2019 at 3:11 AM Michael Ellerman <m...@ellerman.id.au> wrote: >> >> Just one fix, a revert of a commit that was meant to be a minor improvement >> to >> some inline asm, but ended up having no real benefit with GCC and broke >> booting >> 32-bit machines when using Clang. > > Pulled, but whenever there are possible subtle compiler issues I get > nervous, and wonder if the problem was reported to the clang guys?
Yes, sorry I should have included more context. It was actually the Clang Linux folks who noticed it and reported it to us: https://github.com/ClangBuiltLinux/linux/issues/593 There's an LLVM bug filed: https://bugs.llvm.org/show_bug.cgi?id=42762 And I think there's now agreement that the Clang behaviour is not correct, Nick actually sent a revert as well but I already had one queued: https://patchwork.ozlabs.org/patch/1144980/ Arnd identified some work arounds, which we may end up using, but for this cycle we thought it was preferable to just revert this change as it didn't actually change code generation with GCC anyway. cheers
