fixup! [RISCV] Fix missing scaling by LMUL in cost model

simeonkr · simeonkr · commit c77cd418648f · 2024-01-19T16:56:43.000Z
diff --git a/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp b/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
@@ -784,9 +784,8 @@ RISCVTTIImpl::getIntrinsicInstrCost(const IntrinsicCostAttributes &ICA,
   }
   // TODO: add more intrinsic
   case Intrinsic::experimental_stepvector: {
-    unsigned Cost = 1; // vid
     auto LT = getTypeLegalizationCost(RetTy);
-    return (Cost + (LT.first - 1)) * TLI->getLMULCost(LT.second);
+    return LT.first * TLI->getLMULCost(LT.second);
   }
   case Intrinsic::vp_rint: {
     // RISC-V target uses at least 5 instructions to lower rounding intrinsics.

Original file line number	Diff line number	Diff line change
`@@ -784,9 +784,8 @@ RISCVTTIImpl::getIntrinsicInstrCost(const IntrinsicCostAttributes &ICA,`
`784`	`784`	`}`
`785`	`785`	`// TODO: add more intrinsic`
`786`	`786`	`case Intrinsic::experimental_stepvector: {`
`787`		`- unsigned Cost = 1; // vid`
`788`	`787`	`auto LT = getTypeLegalizationCost(RetTy);`
`789`		`- return (Cost + (LT.first - 1)) * TLI->getLMULCost(LT.second);`
	`788`	`+ return LT.first * TLI->getLMULCost(LT.second);`
`790`	`789`	`}`
`791`	`790`	`case Intrinsic::vp_rint: {`
`792`	`791`	`// RISC-V target uses at least 5 instructions to lower rounding intrinsics.`