[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