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

[Jongwon Lee via llvm-commits]

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);
     }
----------------
mzolotukhin wrote:
> 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.


http://reviews.llvm.org/D18237