This is an archive of the discontinued LLVM Phabricator instance.

[MachinePipeliner][NFC] Add some debug log and statistics
ClosedPublic

Authored by jsji on May 20 2019, 2:57 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

jsji created this revision.May 20 2019, 2:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 20 2019, 2:57 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
steven.zhang accepted this revision.May 20 2019, 7:42 PM

LGTM, except some minor comments. However, please hold on for some days, if others have comments.

llvm/lib/CodeGen/MachinePipeliner.cpp
450 ↗(On Diff #200364)

The space should be after the comma instead of before it.

492 ↗(On Diff #200364)

extra space. And 'Not scheduled' ?

508 ↗(On Diff #200364)

Too -> too

This revision is now accepted and ready to land.May 20 2019, 7:42 PM
This revision was automatically updated to reflect the committed changes.