[llvm] [RISCV][CostModel] Updates reduction and shuffle cost (PR #77342)

Luke Lau via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 18 16:02:36 PST 2024


================
@@ -84,8 +84,12 @@ RISCVTTIImpl::getRISCVInstructionCost(ArrayRef<unsigned> OpCodes, MVT VT,
       Cost += VL;
       break;
     }
+    case RISCV::VMV_X_S:
+    case RISCV::VFMV_F_S:
     case RISCV::VMV_S_X:
-      // FIXME: VMV_S_X doesn't use LMUL, the cost should be 1
+    case RISCV::VFMV_S_F:
+      Cost += 1;
+      break;
----------------
lukel97 wrote:

Can we split this off into a separate PR? I think it would be useful to be able to see if this has any effect on the current cost model tests.

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


More information about the llvm-commits mailing list