[PATCH] D18237: [SLPVectorizer] Try to vectorize in the range from MaxVecRegSize to MinVecRegSize
Jongwon Lee via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 23 18:57:32 PDT 2016
JongwonLee marked 2 inline comments as done.
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);
> mssimpso wrote:
> > 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!
Yes. mssimpso is right.
More information about the llvm-commits