[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