[PATCH] D131967: [RISCV] Correct costs for vector ceil/floor/trunc/round

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:340
+                                              ICA.getID(), LT.second))
+        return (LT.first - 1) + Entry->Cost;
+    }
----------------
Shouldn't it be LT.first * Entry->Cost? We'll pay the cost for each piece of the split.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131967/new/

https://reviews.llvm.org/D131967