[PATCH] D18237: [SLPVectorizer] Try to vectorize in the range from MaxVecRegSize to MinVecRegSize

Michael Zolotukhin via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 22 10:03:23 PDT 2016


mzolotukhin added a comment.

> do you think it make sense to push the loops into tryToVectorizeList and canMatchHorizontalReduction?


Actually, it might make sense to keep it as is, the reason being the current form is closer to what we have in `vectorizeStoreChain`. Eventually, it would be cool to merge these two functions together to reduce code duplication. What do you think?

Michael


http://reviews.llvm.org/D18237





More information about the llvm-commits mailing list