[PATCH] D49738: [Dominators] Make RemoveUnreachableBlocks return false if the BasicBlock is already awaiting deletion

Jakub Kuderski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 2 08:36:33 PDT 2018


kuhar added a comment.

Looks fine except for the test IR.



================
Comment at: unittests/Transforms/Utils/Local.cpp:846
+      entry:
+        br label %entry
+      bb0:
----------------
I don't think it's valid to branch to entry (the entry block is supposed to have no predecessors).


https://reviews.llvm.org/D49738





More information about the llvm-commits mailing list