[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