[PATCH] D29252: [NewGVN] Update dominator tree for unreachable blocks.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 28 06:39:41 PST 2017


fhahn added inline comments.


================
Comment at: lib/Transforms/Scalar/NewGVN.cpp:1911
+    for (auto *DI : Children)
+      DT->changeImmediateDominator(DI, DT->getRootNode());
+
----------------
Another possibility would be to add a function to recursively erase all nodes dominated by BB. What do you think?


https://reviews.llvm.org/D29252





More information about the llvm-commits mailing list