diff --git a/llvm/lib/Analysis/ValueTracking.cpp b/llvm/lib/Analysis/ValueTracking.cpp index 2dd671b4ab9e..569ee6b3ea86 100644 --- a/llvm/lib/Analysis/ValueTracking.cpp +++ b/llvm/lib/Analysis/ValueTracking.cpp @@ -3593,14 +3593,23 @@ static bool cannotBeOrderedLessThanZeroImpl(const Value *V, // Unsigned integers are always nonnegative. case Instruction::UIToFP: return true; - case Instruction::FMul: case Instruction::FDiv: - // X * X is always non-negative or a NaN. // X / X is always exactly 1.0 or a NaN. if (I->getOperand(0) == I->getOperand(1) && (!SignBitOnly || cast(I)->hasNoNaNs())) return true; + // Set SignBitOnly for RHS, because X / -0.0 is -Inf (or NaN). + return cannotBeOrderedLessThanZeroImpl(I->getOperand(0), TLI, SignBitOnly, + Depth + 1) && + cannotBeOrderedLessThanZeroImpl(I->getOperand(1), TLI, + /*SignBitOnly*/ true, Depth + 1); + case Instruction::FMul: + // X * X is always non-negative or a NaN. + if (I->getOperand(0) == I->getOperand(1) && + (!SignBitOnly || cast(I)->hasNoNaNs())) + return true; + LLVM_FALLTHROUGH; case Instruction::FAdd: case Instruction::FRem: diff --git a/llvm/test/Transforms/InstSimplify/floating-point-compare.ll b/llvm/test/Transforms/InstSimplify/floating-point-compare.ll index a45e39bd20cb..a32717aed236 100644 --- a/llvm/test/Transforms/InstSimplify/floating-point-compare.ll +++ b/llvm/test/Transforms/InstSimplify/floating-point-compare.ll @@ -999,6 +999,22 @@ define <2 x i1> @known_positive_une_with_negative_constant_splat_vec(<2 x i32> % ret <2 x i1> %cmp } +; TODO: This could fold to true. +define i1 @pr58046(i64 %arg) { +; CHECK-LABEL: @pr58046( +; CHECK-NEXT: [[FP:%.*]] = uitofp i64 [[ARG:%.*]] to double +; CHECK-NEXT: [[MUL:%.*]] = fmul double -0.000000e+00, [[FP]] +; CHECK-NEXT: [[DIV:%.*]] = fdiv double 1.000000e+00, [[MUL]] +; CHECK-NEXT: [[CMP:%.*]] = fcmp oeq double [[DIV]], 0xFFF0000000000000 +; CHECK-NEXT: ret i1 [[CMP]] +; + %fp = uitofp i64 %arg to double + %mul = fmul double -0.000000e+00, %fp + %div = fdiv double 1.000000e+00, %mul + %cmp = fcmp oeq double %div, 0xFFF0000000000000 + ret i1 %cmp +} + define i1 @nonans1(double %in1, double %in2) { ; CHECK-LABEL: @nonans1( ; CHECK-NEXT: ret i1 false