[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.


http://reviews.llvm.org/D18237





More information about the llvm-commits mailing list