[llvm] [AArch64][SVE2] Allow commuting two-input NBSL/BSL2N idioms. (PR #184847)

Ricardo Jesus via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 12 08:52:17 PDT 2026


rj-jesus wrote:

> The transforms LGTM.
> I'm not familiar with `ExpandPseudoInsts` (`renamable`) nor scheduling definitions (Do we even need them here and do they handle the potential `MOVPRFX` ?)

Thanks. I believe the scheduling changes are necessary because the corresponding models have been marked as complete, and the pseudos added here lack default scheduling resources. AFAIU there are no "generic" resources like "WriteVq" that would make sense to associate with the pseudos (we could associate an empty `Sched<[]>`, but that would probably not model the instructions very well). We don't model the potential `MOVPRFX`, but the assumption is that (if emitted) it will be fused with the subsequent instruction.

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


More information about the llvm-commits mailing list