This is an archive of the discontinued LLVM Phabricator instance.

[PlaceSafepoints] Use analysis infrastructure to get dominator tree
ClosedPublic

Authored by reames on May 7 2015, 4:29 PM.

Details

Summary

The old code computed dominators for every loop. This was terribly slow with no good reason. Just use the standard infrastructure for analysis passes.

Diff Detail

Repository
rL LLVM

Event Timeline

reames updated this revision to Diff 25261.May 7 2015, 4:29 PM
reames retitled this revision from to [PlaceSafepoints] Use analysis infrastructure to get dominator tree.
reames updated this object.
reames edited the test plan for this revision. (Show Details)
reames added a reviewer: sanjoy.
reames added a subscriber: Unknown Object (MLST).
sanjoy accepted this revision.May 8 2015, 1:20 AM
sanjoy edited edge metadata.
This revision is now accepted and ready to land.May 8 2015, 1:20 AM
This revision was automatically updated to reflect the committed changes.