[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