[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