From: Zia Ansari Date: Thu, 8 Dec 2016 23:27:40 +0000 (+0000) Subject: [InstSimplify] Add "X / 1.0" to SimplifyFDivInst. X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ea5455c7880a629905c01674eb4d455a2d7abe8b;p=llvm [InstSimplify] Add "X / 1.0" to SimplifyFDivInst. Differential Revision: https://reviews.llvm.org/D27587 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@289153 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/lib/Analysis/InstructionSimplify.cpp b/lib/Analysis/InstructionSimplify.cpp index bbabf2af691..e620e09f5db 100644 --- a/lib/Analysis/InstructionSimplify.cpp +++ b/lib/Analysis/InstructionSimplify.cpp @@ -1127,6 +1127,10 @@ static Value *SimplifyFDivInst(Value *Op0, Value *Op1, FastMathFlags FMF, if (match(Op1, m_Undef())) return Op1; + // X / 1.0 -> X + if (match(Op1, m_FPOne())) + return Op0; + // 0 / X -> 0 // Requires that NaNs are off (X could be zero) and signed zeroes are // ignored (X could be positive or negative, so the output sign is unknown). @@ -4093,6 +4097,8 @@ static Value *SimplifyFPBinOp(unsigned Opcode, Value *LHS, Value *RHS, return SimplifyFSubInst(LHS, RHS, FMF, Q, MaxRecurse); case Instruction::FMul: return SimplifyFMulInst(LHS, RHS, FMF, Q, MaxRecurse); + case Instruction::FDiv: + return SimplifyFDivInst(LHS, RHS, FMF, Q, MaxRecurse); default: return SimplifyBinOp(Opcode, LHS, RHS, Q, MaxRecurse); } diff --git a/test/Transforms/InstSimplify/floating-point-arithmetic.ll b/test/Transforms/InstSimplify/floating-point-arithmetic.ll index ef8adcb1646..21c9fdde150 100644 --- a/test/Transforms/InstSimplify/floating-point-arithmetic.ll +++ b/test/Transforms/InstSimplify/floating-point-arithmetic.ll @@ -62,12 +62,10 @@ define double @fmul_X_1(double %a) { ret double %b } -; FIXME: ; fdiv X, 1.0 ==> X define float @fdiv_x_1(float %a) { ; CHECK-LABEL: @fdiv_x_1( -; CHECK-NEXT: [[RET:%.*]] = fdiv float %a, 1.000000e+00 -; CHECK-NEXT: ret float [[RET]] +; CHECK-NEXT: ret float %a ; %ret = fdiv float %a, 1.0 ret float %ret