[PATCH] D140789: [SLP] Unify GEP cost modeling for load, store and GEP nodes.

[Valeriy Dmitriev via Phabricator via llvm-commits]

vdmitrie created this revision.
vdmitrie added reviewers: ABataev, RKSimon.
vdmitrie added a project: LLVM.
Herald added subscribers: vporpo, hiraditya.
Herald added a project: All.
vdmitrie requested review of this revision.
Herald added subscribers: llvm-commits, pcwang-thead.

Make a separate routine for GEPs cost calculation and make
the approach uniform across load, store and GEP tree nodes.
Additional issue fixed is GEP cost savings were applied twice
for ScatterVectorize nodes (aka gather load) making them look
unrealistically profitable for vectorization.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D140789

Files:
  llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
  llvm/test/Transforms/SLPVectorizer/X86/geps-non-pow-2.ll
  llvm/test/Transforms/SLPVectorizer/X86/remark_gather-load-redux-cost.ll
  llvm/test/Transforms/SLPVectorizer/X86/scatter-vectorize-reused-pointer.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D140789.485727.patch
Type: text/x-patch
Size: 9603 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221230/bb52d9c1/attachment.bin>