[llvm] [RISCV][ISelLowering] Use Zicond for FP selects on Zfinx/Zdinx (PR #169299)

Sam Elliott via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 24 22:27:19 PST 2025


lenary wrote:

Your concerns are right, and that's partly what i wanted to see with the testing. I'm glad you also caught the crash :)

The branch version can also make further savings in longer sequences, where the final pair of `mv`s might not need to be emitted.

I think you should probably skip this optimisation for rv32_zdinx.

Aside: One thing we're pretty useless at with rv32_zdinx is using `FSGNJ.D` to move pairs of registers, as your examples show

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


More information about the llvm-commits mailing list