[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