[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