[PATCH] D128784: [RISCV] Add cost model for vector.reverse mask operation

[WangLian via Phabricator via llvm-commits]

Jimerlife added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:203
+        // Mask operation additionally required extend and truncate
+        return LT.first * 8;
       return LT.first * 6;
----------------
craig.topper wrote:
> Isn't extend 2 instructions, vmv+vmerge?
Yes, you are right. I forget to add vmv. Thank you.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128784