[PATCH] D30744: Improve machine schedulers for in-order processors

Andrew Trick via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 22 10:52:02 PDT 2017

atrick requested changes to this revision.
atrick added a comment.
This revision now requires changes to proceed.


I don't think you need the SingleIssue flag in MCSchedule.h any more.

Also, I don't see the hazard checker logic yet. I think you still need to so this:

- In the scheduler, mustBeginGroup() is a hazard if CurMOps > 0 when isTop() is true. Similarly, mustEndGroup() is the same hazard for !isTop().

It looks like only half the logic is there now--the part where you bump the cycle when finishing a group. Maybe you need a better test case to exercise the hazard checking logic?


More information about the llvm-commits mailing list