[PATCH] D18237: [SLPVectorizer] Try to vectorize in the range from MaxVecRegSize to MinVecRegSize
Chad Rosier via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 22 13:05:40 PDT 2016
mcrosier added a comment.
In http://reviews.llvm.org/D18237#380399, @mzolotukhin wrote:
> > 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?
That's fine by me, Michael.
More information about the llvm-commits