[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