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

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 23 04:57:56 PDT 2019


ABataev added a comment.

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.


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

https://reviews.llvm.org/D57779





More information about the llvm-commits mailing list