[PATCH] D19501: Add LoadStoreVectorizer pass

[Justin Lebar via llvm-commits]

jlebar added a subscriber: jlebar.
jlebar added a comment.

I am very excited about this; please let me know if there's anything I can do to help get it in.


================
Comment at: include/llvm/Transforms/Vectorize.h:147
@@ +146,3 @@
+//
+Pass *createLoadStoreVectorizerPass(unsigned VecRegSize);
+
----------------
Should the VecRegSize be pulled from TTI (at least as a default)?

================
Comment at: lib/Transforms/Vectorize/LoadStoreVectorizer.cpp:88
@@ +87,3 @@
+  /// Checks if there are any memory instructions
+  /// which may be alias betwwen From and To.
+  bool isVectorizable(ArrayRef<Value *> &Chain, BasicBlock::iterator From,
----------------
Typos


http://reviews.llvm.org/D19501