[PATCH] D34135: [LVI] Add initial result to avoid infinite getValueFromCondition recursion

Mikael Holmén via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 4 03:01:55 PDT 2017


uabelho added a comment.

In https://reviews.llvm.org/D34135#883767, @efriedma wrote:

> Update: https://reviews.llvm.org/rL314435 has now landed, so we can use domtree to detect dead blocks.


I saw it was reverted in r314589 but if it makes it into tree again, will that then mean that we can
replace the "trivially dead" checks

pred_empty(BB)

in JumpThreadingPass::ProcessBlock and JumpThreadingPass::runImpl with

!DT->isReachableFromEntry(BB)

and problem is solved?


https://reviews.llvm.org/D34135





More information about the llvm-commits mailing list