[PATCH] D57779: [SLP] Add support for throttling.

[Dinar Temirbulatov via Phabricator via llvm-commits]

dtemirbulatov marked an inline comment as done.
dtemirbulatov added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:7281-7282
+          Cost = V.getTreeCost() + ReductionCost;
+        }
+        if (!Throttling) {
           V.getORE()->emit([&]() {
----------------
dtemirbulatov wrote:
> ABataev wrote:
> > Just `else`?
> We might try partial vectorization without success here and we to report about insufficient cost and break  
We might try partial vectorization without success here and we have to report about insufficient cost and break


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57779/new/

https://reviews.llvm.org/D57779