[PATCH] D136662: [VP][RISCV] Add vp.rint and RISC-V support.

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:518
+    auto LT = getTypeLegalizationCost(RetTy);
+    return Cost + (LT.first - 1);
+  }
----------------
Why is LT.first added? Shouldn't it be multiplied?   stepvector is special because if LT.first > 1, it's a single vid.v and LT.first-1 adds.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136662