[PATCH] D24057: [LoadStoreVectorizer] Change VectorSet to Vector to match head and tail positions. Resolves PR29148.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 30 16:45:22 PDT 2016


jlebar added a comment.

> Is it worth extending this?


Yeah, I think I had the same concern in the original review.

I don't know if it's a worthwhile optimization or not.  But if you can hack something together, it should be easy to run an experiment -- compile TensorFlow (or Eigen or Thrust) and use the statistics we already have to count how many extra vectorization opportunities we find.


https://reviews.llvm.org/D24057





More information about the llvm-commits mailing list