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

Jongwon Lee via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 22 03:31:24 PDT 2016


JongwonLee added a comment.

Data for compile-time and execution-time is attached. 
This patch doesn't change the default value of MinVecRegSize, so it does not affect the performance.
(On top of this patch, another patch is needed to change the default value of MinVecRegSize.)
In commercial benchmark, I found that clang lost the chance of 64-bit SLP vectorization in AArch64.
Although there would be side effect of extension of 64-bit SLP vectorization, the compiler should handle 64-bit SLP vectorization in AArch64. The side effect, if any, should be handled with another patch.


http://reviews.llvm.org/D18237





More information about the llvm-commits mailing list