[PATCH] D19501: Add LoadStoreVectorizer pass

[Michael Zolotukhin via llvm-commits]

mzolotukhin added a comment.

Hi Matt,

I think it's ok to have it in a separate pass, but we really need to try to factor out common parts. For instance, we definitely have `propagateMetadata` and `isConsecutiveAccess` in SLP.

Thanks,
Michael


================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:518
@@ +517,3 @@
+
+  for (auto I = Map.begin(), E = Map.end(); I != E; ++I) {
+    unsigned size = I->second.size();
----------------
Can it be a range loop?


http://reviews.llvm.org/D19501