[PATCH] D132590: [SLP] Try to match reductions first in a vector build sequence.

[Valeriy Dmitriev via Phabricator via llvm-commits]

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

This patch changes order of searching for reductions vs other vectorization possibility.
The idea is if we do not match a reduction it won't be harmful for further attempts to
find vectorizable operations. But doing it in the opposite order we have good chance
to ruin possibility to match a reduction. While attempting to vectorize reductions
on operands of a vector build sequence we also don't want to try vectorizing binary
operations this early as 2-way vectorization may effectively prohibit wider ones
leading to producing less effective code.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D132590

Files:
  llvm/include/llvm/Transforms/Vectorize/SLPVectorizer.h
  llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
  llvm/test/Transforms/SLPVectorizer/X86/redux-feed-buildvector.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D132590.455322.patch
Type: text/x-patch
Size: 23003 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220824/3c454c55/attachment.bin>