[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: lib/Transforms/Vectorize/SLPVectorizer.cpp:126
+static cl::opt<int>
+    MaxCostsRecalculations("slp-throttling-budget", cl::init(64), cl::Hidden,
+                           cl::desc("Limit the total number of nodes for cost "
----------------
ABataev wrote:
> Why 64? Did you try to estimate effect of this change on the compilation time?
oh, That just from my synthetic examples. I have noticed double on SLP time with 100+ tree nodes examples.


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

https://reviews.llvm.org/D57779