[llvm] [RISCV] Add Zilsd to RISCVMergeBaseOffset. (PR #140157)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Fri May 16 11:34:27 PDT 2025


topperc wrote:

> @topperc i guess if this pass is early enough we'd have an insert_subreg. You're right that the check you pointed out is the one that I wondered if it needed generalising to some kind of "do these two alias". My main thought is that if this was a problem, we'd have already seen it with rv32Zdinx loads and stores, which i think are already supported.

rv32zdinx doesn't have any loads/stores other than the Pseudo that we expand later. And those pseudos are not recognized by this pass. And we stopped using those pseudos in isel earlier this week.

https://github.com/llvm/llvm-project/pull/140157


More information about the llvm-commits mailing list