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

[Nemanja Ivanovic via Phabricator via llvm-commits]

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

LGTM.



================
Comment at: lib/Target/PowerPC/PPCSubtarget.cpp:187
+  // The GenericScheduler that we use defaults to scheduling bottom up only.
+  // We want to schedule from both the top and the bottom and so we set
+  // OnlyBottomUp to false.
----------------
I agree with Eric, please add the reason. Perhaps `We want to do bi-directional scheduling since it provides a more balanced schedule leading to better performance.`


https://reviews.llvm.org/D48663