[PATCH] D34651: [Dominators] Use a custom DFS implementation

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 16:34:39 PDT 2017


dberlin added a comment.

So, just to note: It's generally nonsense to try to calculate the root list prior to DFS (and a source of bugs in the current postdominator tree).  By nonsense i mean "impossible".
You need to see what's reachable to know what ends up needing to be connected to a virtual root.


https://reviews.llvm.org/D34651





More information about the llvm-commits mailing list