This is an archive of the discontinued LLVM Phabricator instance.

[NewPM] Port MachineDominatorTree analysis to the new PM.
AcceptedPublic

Authored by czhang on Jul 3 2019, 6:36 PM.

Details

Reviewers
nhaehnle

Event Timeline

czhang created this revision.Jul 3 2019, 6:36 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 3 2019, 6:37 PM

Could you clang-format? There seem to be lines that exceed 80 cols.

fhahn added a subscriber: fhahn.Jul 5 2019, 7:19 AM
czhang updated this revision to Diff 208559.Jul 8 2019, 5:50 PM

rebase & clang-format

nhaehnle accepted this revision.Jul 12 2019, 6:08 AM

You should probably wait a bit in case somebody else wants to chime in, but this looks good to me.

This revision is now accepted and ready to land.Jul 12 2019, 6:08 AM

Thanks for reviewing! Since this change is dependent on the first patch in my series fitting the new pass manager in CodeGen here: https://reviews.llvm.org/D64179, would you mind taking a look at that as well? The entire patch series: https://reviews.llvm.org/D63890