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

[Eric Christopher via Phabricator via llvm-commits]

echristo added a comment.





================
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.
----------------
nemanjai wrote:
> 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.`
Yeah... though that's not even really a reason, just describing the effect. :\


https://reviews.llvm.org/D48663