[llvm-commits] [llvm] r112270 - in /llvm/trunk: lib/Analysis/LazyValueInfo.cpp lib/Transforms/Scalar/JumpThreading.cpp test/Transforms/JumpThreading/basic.ll test/Transforms/JumpThreading/lvi-load.ll

[Owen Anderson]

On Aug 27, 2010, at 10:21 AM, Chris Lattner wrote:

> Testcase?   I don't think this fixes PR5652, which isn't about deleting a dead condition, it is about *changing* a dominating condition to make one dead.

The test for this is folded into the changes I made to basic.ll, though I can also add an explicit one.

As to PR5652, from the commentary:

The reduced issue is basically:
if (x < 10)
  if (c)
    if (x < 21)
this can be optimized to:
if (x < 21)
  if (c)


This patch does handle that case, and the testcase in the OP (modulo the the comparisons being backwards as the reporter acknowledged).

--Owen