[PATCH] D62751: [Utils] Clean another duplicated util method.
    Alina Sbirlea via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Jun  6 12:21:21 PDT 2019
    
    
  
asbirlea added a comment.
It appears the difference is in the way the DT updates are done.
+ at kuhar, @NutshellySima: Could you please take a look at the way the updates are done in MergeBlockIntoPredecessor vs (the removed) foldBlockIntoPredecessor? I tried to use a Lazy strategy and flush after doing all the potential merges, but the regression is still there. Can we get the same performance with the DomTreeUpdater here?
Repository:
  rL LLVM
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62751/new/
https://reviews.llvm.org/D62751
    
    
More information about the llvm-commits
mailing list