[PATCH] D38960: [Local] Fix a bug in the domtree update logic for MergeBasicBlockIntoOnlyPred.
    Balaram Makam via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Oct 16 09:53:41 PDT 2017
    
    
  
bmakam added a comment.
In https://reviews.llvm.org/D38960#898701, @kuhar wrote:
> > For some irreducible CFG the domtree nodes might be dead, do not update domtree for dead nodes.
>
> Can you show some example of such CFG and add a comment explaining this check?
>
> DomTree::getNode return nullptr only for forward-unreachable CFG nodes.
I found this bug while testing https://reviews.llvm.org/D37343. When https://reviews.llvm.org/D37343 is applied, test/Transforms/CodeGenPrepare/2008-11-24-RAUW-Self.ll will crash because of this.
https://reviews.llvm.org/D38960
    
    
More information about the llvm-commits
mailing list