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

Fri Aug 23 05:06:49 PDT 2019

dtemirbulatov added a comment.

In D57779#1642591 <https://reviews.llvm.org/D57779#1642591>, @ABataev wrote:

> In D57779#1642570 <https://reviews.llvm.org/D57779#1642570>, @dtemirbulatov wrote:
>
> > > Some of the results are worse than without the patch. Can we investigate why?
> >
> > yes, I will focus on 401.bzip2, 429.mcf. but this SLP throttling supposes be off by default and it on here just to show the impact on tests.
>
>
> Even if it off by default, I don't think it is a good idea to have the code that may lead to regressions when we could avoid that.

yes, ok.

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

https://reviews.llvm.org/D57779