[PATCH] D57316: [LoopSimplifyCFG] Fix DT update strategy

Max Kazantsev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 28 01:03:41 PST 2019


mkazantsev created this revision.
mkazantsev added reviewers: anna, asbirlea, skatkov, reames.

DT Updater sometimes does not update nodes in unreachable subtrees
(unreachable in terms of DomTree). When we use eager update strategy,
we can end up not making some updates that need to be made because
previous update break the tree.

This patch fixes this behavior by using lazy update strategy in DTU. We only
make all updates in the end (and when we need to form LCSSA after dead
exit fixup and before folding).


https://reviews.llvm.org/D57316

Files:
  lib/Transforms/Scalar/LoopSimplifyCFG.cpp
  test/Transforms/LoopSimplifyCFG/constant-fold-branch.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57316.183808.patch
Type: text/x-patch
Size: 7341 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190128/bbd334c3/attachment.bin>


More information about the llvm-commits mailing list