[llvm] [RISCV] Make fixed-point instructions commutable (PR #90035)

Luke Lau via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 25 06:07:49 PDT 2024


================
@@ -2718,7 +2720,7 @@ multiclass VPseudoVGTR_VV_VX_VI<Operand ImmType = simm5, string Constraint = "">
 multiclass VPseudoVSALU_VV_VX_VI<Operand ImmType = simm5, string Constraint = ""> {
   foreach m = MxList in {
     defvar mx = m.MX;
-    defm "" : VPseudoBinaryV_VV<m, Constraint>,
+    defm "" : VPseudoBinaryV_VV<m, Constraint, Commutable=1>,
----------------
lukel97 wrote:

Maybe add a Commutable argument to VPseudoVSALU_VV_VX_VI and explicitly set it in the PseudoVSADD[U] defms? So it's more obvious that only the add pseudos are commutable

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


More information about the llvm-commits mailing list