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

[Andrew Trick via Phabricator via llvm-commits]

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

Thanks.

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?


https://reviews.llvm.org/D30744