[PATCH] D48383: [Dominators] Add the DomTreeUpdater class

Chijun Sima via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 23 10:28:37 PDT 2018


NutshellySima updated this revision to Diff 152596.
NutshellySima marked an inline comment as done.
NutshellySima added a comment.

1. Address comments.
2. Remove recalculateDomTree() and recalculatePostDomTree().
3. Remove the "removeInvalidUpdate" parameter of insertEdge() and deleteEdge() and update docs to reflect this change.
4. Rename recalculateAll() to recalculate() (It can recalculate all holding trees).
5. Rename flushDomTree()/flushPostDomTree() to getDomTree()/getPostDomTree() because the functions indeed return the updated trees.


https://reviews.llvm.org/D48383

Files:
  include/llvm/IR/DomTreeUpdater.h
  lib/IR/CMakeLists.txt
  lib/IR/DomTreeUpdater.cpp
  unittests/IR/CMakeLists.txt
  unittests/IR/DomTreeUpdaterTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48383.152596.patch
Type: text/x-patch
Size: 51147 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180623/d1480d7a/attachment.bin>


More information about the llvm-commits mailing list