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

[Matthew Simpson via llvm-commits]

mssimpso 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.


http://reviews.llvm.org/D18237