Index: lib/Transforms/InstCombine/InstCombineCompares.cpp =================================================================== --- lib/Transforms/InstCombine/InstCombineCompares.cpp +++ lib/Transforms/InstCombine/InstCombineCompares.cpp @@ -3173,6 +3173,22 @@ return Res; } + // (icmp sgt smin(PosA, B) 0) -> (icmp sgt B 0) + if (CI->isZero() && I.getPredicate() == ICmpInst::ICMP_SGT) + if (auto *SI = dyn_cast(Op0)) { + SelectPatternResult SPR = matchSelectPattern(SI, A, B); + switch (SPR.Flavor) { + default: break; + case SPF_SMIN: + if (isKnownNonNegative(A, DL)) + return new ICmpInst(I.getPredicate(), B, CI); + if (isKnownNonNegative(B, DL)) + return new ICmpInst(I.getPredicate(), A, CI); + break; + }; + } + + // The following transforms are only 'worth it' if the only user of the // subtraction is the icmp. if (Op0->hasOneUse()) { Index: test/Transforms/InstCombine/min-positive.ll =================================================================== --- test/Transforms/InstCombine/min-positive.ll +++ test/Transforms/InstCombine/min-positive.ll @@ -0,0 +1,23 @@ +; RUN: opt -S -instcombine < %s | FileCheck %s + +@g = external global i32 + +define i1 @test(i32 %other) { +; CHECK-LABEL: @test +; CHECK: %test = icmp sgt i32 %other, 0 + %positive = load i32, i32* @g, !range !{i32 0, i32 2048} + %cmp = icmp slt i32 %positive, %other + %sel = select i1 %cmp, i32 %positive, i32 %other + %test = icmp sgt i32 %sel, 0 + ret i1 %test +} + +define i1 @test2(i32 %other) { +; CHECK-LABEL: @test2 +; CHECK: %test = icmp sgt i32 %other, 0 + %positive = load i32, i32* @g, !range !{i32 0, i32 2048} + %cmp = icmp slt i32 %other, %positive + %sel = select i1 %cmp, i32 %other, i32 %positive + %test = icmp sgt i32 %sel, 0 + ret i1 %test +}