[PATCH] D48967: [Dominators] Convert existing passes and utils to use the DomTreeUpdater class

Chijun Sima via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 11 10:32:11 PDT 2018


NutshellySima marked 21 inline comments as done.
NutshellySima added a comment.

Marked as done because this problem should be addressed separately.
The problem:
Currently `DTU` assumes updates submitted after `getDomTree()/getPostDomTree()/flush()` won't affect the updates submitted before it.
More specifically, it assumes that the caller should not submit updates that already knew by the DT/PDT.
This problem can cause bugs when aggressive passes like `jump-threading` try to preserve PDT in the future.


https://reviews.llvm.org/D48967





More information about the llvm-commits mailing list