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.
Details
Details
Diff Detail
Diff Detail
- Repository
- rL LLVM
Paths
| Differential D31818
[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.
Diff Detail
Event TimelineThis revision is now accepted and ready to land.Apr 10 2017, 11:09 AM Closed by commit rL299893: [MemorySSA] We don't need to compute dominator levels anymore. (authored by davide). · Explain WhyApr 10 2017, 3:57 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 94753 llvm/trunk/include/llvm/Transforms/Utils/MemorySSA.h
llvm/trunk/lib/Transforms/Utils/MemorySSA.cpp
|