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

[Stefan Pintilie via Phabricator via llvm-commits]

stefanp added a comment.

Hi Eric,

We set the generic scheduler to bi-directional scheduling back in 2013. It was at the time when the generic machine scheduler was added to PowerPC.  I can only assume that when we did this we ran a set of benchmarks and determined that this is the best option based on that.
However, what I can do now is run a round of SPEC with Top-Of-Trunk and confirm that we still do better in the bi-directional case. (Or at least not worse.)

Does that sound reasonable?


https://reviews.llvm.org/D48663