[PATCH] D153570: [CostModel][SLP] Use getGEPCost AccessTy in more places

Luke Lau via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 22 09:28:46 PDT 2023


luke created this revision.
luke added reviewers: reames, ABataev, nikic.
Herald added subscribers: asb, vporpo, pmatos, StephenFan, frasercrmck, luismarques, apazos, sameer.abuasal, s.egerton, Jim, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, niosHD, sabuasal, simoncook, johnrusso, rbar, hiraditya, arichardson.
Herald added a project: All.
luke requested review of this revision.
Herald added subscribers: llvm-commits, wangpc, MaskRay.
Herald added a project: LLVM.

As a follow up from D149889 <https://reviews.llvm.org/D149889>, this patch passes in an explicit AccessTy
in SLP (and getPointersChainCost) to get more accurate GEP costs.

It also as adds a helper function to extract the access type of an
already materalized GEP and pass that to getGEPCost, provided the GEP
has exactly one user.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D153570

Files:
  llvm/include/llvm/Analysis/TargetTransformInfo.h
  llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
  llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
  llvm/lib/Transforms/Scalar/NaryReassociate.cpp
  llvm/lib/Transforms/Scalar/StraightLineStrengthReduce.cpp
  llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
  llvm/test/Transforms/SLPVectorizer/RISCV/gep.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D153570.533653.patch
Type: text/x-patch
Size: 6801 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230622/572a0475/attachment.bin>


More information about the llvm-commits mailing list