[PATCH] D48663: [Power9] Code Cleanup - Remove needsAggressiveScheduling()

[Hal Finkel via Phabricator via llvm-commits]

hfinkel added a comment.

In https://reviews.llvm.org/D48663#1152519, @stefanp wrote:

> Hi Hal, Eric,
>
> I ran SPEC2017 on a https://reviews.llvm.org/P9 machine and difference between bi-directional and bottom up scheduling was very small overall. The bi-directional scheduling continues to be a bit better with 
>  xalancbmk - 0.5% better on bi-directional
>  leela - 2.0% better on bi-directional
>
> Changes for all of the other benchmarks were too small.


Sounds good. Thanks for checking. I recommend keeping it bi-directional.


https://reviews.llvm.org/D48663