The majority of the cost appears to be forming the indices vector.
This is my first costing patch, so please check for all the silly mistakes.
Paths
| Differential D127141
[RISCV] Add cost model for reverse shuffle ClosedPublic Authored by reames on Jun 6 2022, 11:54 AM.
Details Summary The majority of the cost appears to be forming the indices vector. This is my first costing patch, so please check for all the silly mistakes.
Diff Detail
Event TimelineComment Actions FYI, there was already another patch for this D125856, but it has some other things in it and I don't believe the number.
Comment Actions
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.
This revision is now accepted and ready to land.Jun 8 2022, 10:13 PM This revision was landed with ongoing or failed builds.Jun 9 2022, 7:22 AM Closed by commit rG0e29a80fdc5e: [RISCV] Add cost model for reverse shuffle (authored by reames). · Explain Why This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 435539 llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
llvm/test/Analysis/CostModel/RISCV/rvv-shuffle.ll
llvm/test/Transforms/LoopVectorize/RISCV/riscv-vector-reverse.ll
|
rli -> srli?