[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