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

[Michael Zolotukhin via llvm-commits]

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);
     }
----------------
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!


http://reviews.llvm.org/D18237