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

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

In D131967#3735818 <https://reviews.llvm.org/D131967#3735818>, @craig.topper wrote:

> Looking at my patch, I thought I increased the number of instructions for every version of ceil and floor, but the cost from your previous revision went from 16 to 15. Am I confused?

As we discussed on that review, you're patch ended up removing 2 vsetvlis.  Since I was only discounting 1 of the original three, it's still a net savings of instructions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131967