[PATCH] D147000: [RISCV] Cost model for general case of single vector permute

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 27 13:46:30 PDT 2023


reames created this revision.
reames added reviewers: luke, craig.topper, asb.
Herald added subscribers: jobnoorman, VincentWu, vkmr, frasercrmck, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, bollu, simoncook, johnrusso, rbar, hiraditya, arichardson, mcrosier.
Herald added a project: All.
reames requested review of this revision.
Herald added subscribers: pcwang-thead, eopXD, MaskRay.
Herald added a project: LLVM.

The cost model was not accounting for the fact that we can generate vrgather + an index expression.

One thing to call out - I did not model the difference between vrgather and vrgatherei16.  The result is the constant pool cost can be slightly understated on RV32.  I'm not sure if we care, if we do, I can add in the handling for that case.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D147000

Files:
  llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
  llvm/test/Analysis/CostModel/RISCV/shuffle-interleave.ll
  llvm/test/Transforms/LoopVectorize/RISCV/interleaved-accesses.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D147000.508785.patch
Type: text/x-patch
Size: 10807 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230327/4cb75d6a/attachment.bin>


More information about the llvm-commits mailing list