[PATCH] D14829: [SLP] Vectorize gather-like idioms ending at non-consecutive loads.

[hfinkel@anl.gov via llvm-commits]

hfinkel added a comment.

> I don't think the expression trees this patch affects are necessarily limited in size.


I agree; these patterns can occur with reasonable complexity in practice.

Have you run this on the test suite, did you find any statistically-significant compile-time impact?


================
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:4294
@@ +4293,3 @@
+    // pursue it further in a bottom-up phase. It should have already been
+    // vectorized if prpofitable. The code below gives up if a single pair of
+    // consecutive accesses is found.
----------------
prpofitable (typo)


http://reviews.llvm.org/D14829