This is an archive of the discontinued LLVM Phabricator instance.

[CodeGen] Add dependency printer
ClosedPublic

Authored by evandro on Jul 7 2017, 2:27 PM.

Details

Summary

Add SDep printer to make debugging sessions more productive.

Diff Detail

Repository
rL LLVM

Event Timeline

evandro created this revision.Jul 7 2017, 2:27 PM
evandro updated this revision to Diff 105698.Jul 7 2017, 2:35 PM
MatzeB accepted this revision.Jul 10 2017, 7:33 PM

LGTM

llvm/lib/CodeGen/ScheduleDAG.cpp
71 ↗(On Diff #105698)

May want to pass in TargetRegisterInfo* instead of ScheduleDAG if that is the only thing used from the ScheduleDAG.

This revision is now accepted and ready to land.Jul 10 2017, 7:33 PM
evandro added inline comments.Jul 11 2017, 11:06 AM
llvm/lib/CodeGen/ScheduleDAG.cpp
71 ↗(On Diff #105698)

I don't feel strongly about this, but I used ScheduleDAG to keep the method similar to the one in SUnit.

evandro updated this revision to Diff 106080.Jul 11 2017, 12:41 PM
evandro marked an inline comment as done.
This revision was automatically updated to reflect the committed changes.