[llvm] [RISCV][P-ext] Merge paired pli/plui in RISCVMoveMerger (PR #202566)

via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 9 21:54:33 PDT 2026


sihuan wrote:

> > LGTM.
> > I guess i do find it interesting that we end up with `pli.b rN, imm; pli.b rN+1, imm` rather than e.g. `pli.b rN, imm; {c.}mv rN+1, rN`. I don't think it's a problem this isn't supported, if isel never actually creates that sequence though (but maybe it should?)
> 
> We have pli.b marked as rematerializable so that's probably why it doesn't do the mv.

Thanks for the context. Lines up with this pass: rematerialization gives 2x `pli.b` -> `pli.db`.
Planning to merge if there's nothing else.

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


More information about the llvm-commits mailing list