[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 10:05:53 PDT 2017
    
    
  
bmakam added a comment.
In https://reviews.llvm.org/D38960#898713, @kuhar wrote:
> I think that this deserves a comment with a (partial) example, otherwise the change looks suspicious.
Fair point. All the users of MergeBasicBlockIntoOnlyPred in the tree, do not pass DT to the function, so it is never exposed. Is there a way to test this in tree by passing DT to MergeBasicBlockIntoOnlyPred?
https://reviews.llvm.org/D38960
    
    
More information about the llvm-commits
mailing list