This is an archive of the discontinued LLVM Phabricator instance.

[MemorySSA] Remove dominator umbering
ClosedPublic

Authored by davide on Apr 7 2017, 10:51 AM.

Details

Summary

Apparently this was originally used in the phi placement algorithm, but now it's pretty much dead.
I wasn't sure if you planned to use for something else in the future, hence this review.

Diff Detail

Repository
rL LLVM

Event Timeline

davide created this revision.Apr 7 2017, 10:51 AM
dberlin accepted this revision.Apr 10 2017, 11:09 AM

No, this is definitely dead now i believe :)

This revision is now accepted and ready to land.Apr 10 2017, 11:09 AM
This revision was automatically updated to reflect the committed changes.