InstCombine: Set operands instead of creating new call

llvm-svn: 291612
This commit is contained in:
Matt Arsenault 2017-01-10 23:17:52 +00:00
parent fdb78f8bae
commit 3f509042b0
1 changed files with 6 additions and 10 deletions

View File

@ -1599,21 +1599,17 @@ Instruction *InstCombiner::visitCallInst(CallInst &CI) {
// fma fneg(x), fneg(y), z -> fma x, y, z
if (match(Src0, m_FNeg(m_Value(LHS))) &&
match(Src1, m_FNeg(m_Value(RHS)))) {
CallInst *NewCall = Builder->CreateCall(II->getCalledFunction(),
{LHS, RHS, II->getArgOperand(2)});
NewCall->takeName(II);
NewCall->copyFastMathFlags(II);
return replaceInstUsesWith(*II, NewCall);
II->setArgOperand(0, LHS);
II->setArgOperand(1, RHS);
return II;
}
// fma fabs(x), fabs(x), z -> fma x, x, z
if (match(Src0, m_Intrinsic<Intrinsic::fabs>(m_Value(LHS))) &&
match(Src1, m_Intrinsic<Intrinsic::fabs>(m_Value(RHS))) && LHS == RHS) {
CallInst *NewCall = Builder->CreateCall(II->getCalledFunction(),
{LHS, LHS, II->getArgOperand(2)});
NewCall->takeName(II);
NewCall->copyFastMathFlags(II);
return replaceInstUsesWith(*II, NewCall);
II->setArgOperand(0, LHS);
II->setArgOperand(1, RHS);
return II;
}
// fma x, 1, z -> fadd x, z