[PATCH] D18237: [SLPVectorizer] Try to vectorize in the range from MaxVecRegSize to MinVecRegSize
Michael Zolotukhin via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 23 14:29:11 PDT 2016
mzolotukhin added inline comments.
Comment at: lib/Transforms/Vectorize/SLPVectorizer.cpp:4544
@@ -4557,4 +4543,3 @@
unsigned Len = std::min<unsigned>(CE - CI, 16);
- Changed |= vectorizeStores(makeArrayRef(&it->second[CI], Len),
- -SLPCostThreshold, R);
+ Changed |= vectorizeStores(makeArrayRef(&it->second[CI], Len), R);
> mzolotukhin wrote:
> > `SLPCostThreshold` disappeared after this change. Was it intentional?
> SLPCostThreshold is a command line option, so it doesn't need to be passed as a function parameter.
Ah, right, thanks for pointing that out!
More information about the llvm-commits