[PATCH] D39606: [SLP] Fix PR23510: Try to find best possible vectorizable stores.
Alexey Bataev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 7 06:06:06 PST 2017
ABataev added inline comments.
================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:4334
// If a store has multiple consecutive store candidates, search Stores
- // array according to the sequence: from i+1 to e, then from i-1 to 0.
+ // array according to the sequence: Idx-1, Idx+1, Idx-2, Idx+2, ...
// This is because usually pairing with immediate succeeding or preceding
----------------
RKSimon wrote:
> > Idx-1, Idx+1, Idx-2, Idx+2, ...
>
> Idx+2 ? Fix this sequence
>
>
I don't understand, what's wrong with the sequence?
https://reviews.llvm.org/D39606
More information about the llvm-commits
mailing list