[PATCH] D60897: [SLP] Look-ahead operand reordering heuristic.

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 28 11:13:17 PDT 2019


phosek added a comment.

In D60897#1562519 <https://reviews.llvm.org/D60897#1562519>, @vporpo wrote:

> I think There are two possible causes for the compilation time increase:
>
> 1. Line 901 : We can restrict the number of operands to a max of 2
> 2. Line 820:  We can restrict the visited users to a ma x of 2. I can either create a quick patch, or I can revert it. Either is fine.


If you can create a quick patch, I'd be happy to try it locally to see if it addresses the problem.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60897





More information about the llvm-commits mailing list