[PATCH] D35851: [Dominators] Include infinite loops in PostDominatorTree

Daniel Berlin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 15:35:45 PDT 2017


dberlin added inline comments.


================
Comment at: lib/Transforms/Scalar/ADCE.cpp:265
+    if (isa<ReturnInst>(Info.Terminator)) {
+      DEBUG(dbgs() << "post-dom root child is not a return: " << BB->getName()
                    << '\n';);
----------------
kuhar wrote:
> @dberlin: shouldn't this be:
> > post-dom root child *is* a return
> 
> ?
> 
> 
yes, it should be.


https://reviews.llvm.org/D35851





More information about the llvm-commits mailing list