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

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
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



More information about the llvm-commits mailing list