Index: lib/Transforms/Scalar/SCCP.cpp =================================================================== --- lib/Transforms/Scalar/SCCP.cpp +++ lib/Transforms/Scalar/SCCP.cpp @@ -928,25 +928,17 @@ NonOverdefVal = &V2State; if (NonOverdefVal) { - if (NonOverdefVal->isUnknown()) { - // Could annihilate value. - if (I.getOpcode() == Instruction::And) - markConstant(IV, &I, Constant::getNullValue(I.getType())); - else if (VectorType *PT = dyn_cast(I.getType())) - markConstant(IV, &I, Constant::getAllOnesValue(PT)); - else - markConstant(IV, &I, - Constant::getAllOnesValue(I.getType())); + if (NonOverdefVal->isUnknown()) return; - } if (I.getOpcode() == Instruction::And) { // X and 0 = 0 if (NonOverdefVal->getConstant()->isNullValue()) return markConstant(IV, &I, NonOverdefVal->getConstant()); } else { + // X or -1 = -1 if (ConstantInt *CI = NonOverdefVal->getConstantInt()) - if (CI->isAllOnesValue()) // X or -1 = -1 + if (CI->isAllOnesValue()) return markConstant(IV, &I, NonOverdefVal->getConstant()); } } Index: test/Transforms/SCCP/logical-nuke.ll =================================================================== --- test/Transforms/SCCP/logical-nuke.ll +++ test/Transforms/SCCP/logical-nuke.ll @@ -1,9 +1,31 @@ -; RUN: opt < %s -sccp -S | grep "ret i32 0" +; RUN: opt < %s -sccp -S | FileCheck %s ; Test that SCCP has basic knowledge of when and/or nuke overdefined values. +; CHECK-LABEL: test +; CHECK: ret i32 0 define i32 @test(i32 %X) { - %Y = and i32 %X, 0 ; [#uses=1] - ret i32 %Y + %Y = and i32 %X, 0 + ret i32 %Y } +; CHECK-LABEL: test2 +; CHECK: ret i32 -1 +define i32 @test2(i32 %X) { + %Y = or i32 -1, %X + ret i32 %Y +} + +; CHECK-LABEL: test3 +; CHECK: ret i32 0 +define i32 @test3(i32 %X) { + %Y = and i32 undef, %X + ret i32 %Y +} + +; CHECK-LABEL: test4 +; CHECK: ret i32 -1 +define i32 @test4(i32 %X) { + %Y = or i32 %X, undef + ret i32 %Y +}