[llvm] [RISCV][CostModel] Updates reduction and shuffle cost (PR #77342)

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 30 07:17:09 PST 2024


================
@@ -482,11 +481,10 @@ InstructionCost RISCVTTIImpl::getShuffleCost(TTI::ShuffleKind Kind,
       //   vmsne.vi  v0, v8, 0
 
       return LT.first *
-             (TLI->getLMULCost(LT.second) + // FIXME: this should be 1 for andi
----------------
preames wrote:

If the index is non-zero, then there's a vslide needed in the current sequence which does scale with LMUL.

So, this change is wrong for non-zero index.  

https://github.com/llvm/llvm-project/pull/77342


More information about the llvm-commits mailing list