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

[Philip Reames via Phabricator via llvm-commits]

reames added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:340
+                                              ICA.getID(), LT.second))
+        return (LT.first - 1) + Entry->Cost;
+    }
----------------
craig.topper wrote:
> Shouldn't it be LT.first * Entry->Cost? We'll pay the cost for each piece of the split.
Yep, I copied this from another block of code and apparently didn't give it enough thought.  Will fix.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131967