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

Matthew Simpson via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 27 04:22:09 PDT 2016


mssimpso added a comment.

Hey Jongwon,

I think it's common to just paste in the most significant execution and compile time improvements and regressions. It's not necessary to report every data point.

Even though you're not changing the value of MinVecRegSize here, I think it makes sense to evaluate this patch and http://reviews.llvm.org/D19151 together so we can see the effect (particularly in compile time) that looping over potential register sizes will have.


http://reviews.llvm.org/D18237





More information about the llvm-commits mailing list