[PATCH] D19501: Add LoadStoreVectorizer pass

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Wed May 25 13:58:09 PDT 2016

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,


More information about the llvm-commits mailing list