[PATCH] D158062: [RISCV] Teach RISCVMergeBaseOffset to handle inline asm

Thu Aug 31 13:48:29 PDT 2023

craig.topper added a comment.

In D158062#4632461 <https://reviews.llvm.org/D158062#4632461>, @nickdesaulniers wrote:

> @samitolvanen is reporting that this is causing breakage for us in the Linux kernel: https://github.com/ClangBuiltLinux/linux/issues/1928.
>
> Please consider whether this can be fixed forward quickly, or should be reverted. Otherwise I will revert shortly.

I think the problem is that atomic instructions don't have an immediate offset field so they are different than other instructions. I'm not sure we can generically do this patch for the "m" constraint. We should revert.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D158062/new/

https://reviews.llvm.org/D158062