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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 28 09:58:38 PDT 2019


RKSimon reopened this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

In D60897#1562427 <https://reviews.llvm.org/D60897#1562427>, @phosek wrote:

> This change has caused a massive increase in build times when using LTO. On my workstations when building Clang toolchain itself with LTO, the build time has increased from 50 minutes to 5+ hours. On our continuous builders, we seen timeouts everywhere because they aren't able to finish within the allocated time (5 hours). Would it be possible to revert this change?


Reversion makes sense to me - @phosek are you in position to profile it to see where the time is going in SLP please?


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