[PATCH] D127141: [RISCV] Add cost model for reverse shuffle

[Philip Reames via Phabricator via llvm-commits]

reames added a comment.

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

> FYI, there was already another patch for this D125856 <https://reviews.llvm.org/D125856>, but it has some other things in it and I don't believe the number.

As you note, that change has an assortment of things scattered in it.  This one is deliberately minimal.  That one can be rebased or abandoned as needed.



================
Comment at: llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp:197
+      //   csrr a0, vlenb
+      //   rli a0, a0, 3
+      //   addi a0, a0, -1
----------------
craig.topper wrote:
> rli -> srli?
Yep, typo.  Will fix.


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

https://reviews.llvm.org/D127141