[PATCH] D43329: [SystemZ, MachineScheduler] Refactor GenericScheduler::tryCandidate() to reuse parts in a new SystemZ scheduling strategy.

[Jonas Paulsson via Phabricator via llvm-commits]

jonpa added a comment.

In https://reviews.llvm.org/D43329#1064275, @fhahn wrote:

> Hi Jonas,
>
> are you planning on committing this patch in the near future? I suppose you are waiting until @uweigand  had a look at the SystemZ specific changes?
>
> Recently, a few people have been looking into tweaking scheduler heuristics, and after this change goes in we could easily add an experimental scheduler for AArch64 to experiment with tweaking heuristics.
>
> Thanks again for putting up the patch!
>
> Florian


I committed the common code parts as r329884. We are waiting a bit with this for SystemZ, since we are having second thoughts that we probably should enable bi-directional scheduling before enabling tryLatency() in a specialized case (without bi-directional the second tryLatency() call is never made).

Thanks for review.


https://reviews.llvm.org/D43329