shylie/llvm-project
1
0
Fork 0
Code Issues Pull Requests Actions 6 Packages Projects Releases Wiki Activity

[InstCombine] use CreateWithCopiedFlags to reduce code; NFCI

Browse Source 
Also, move the folds with constants closer to make it easier to follow. 

llvm-svn: 325541
This commit is contained in:
Sanjay Patel 2018-02-19 23:09:03 +00:00
parent 2e8a8629d6
commit 2816560b2c
1 changed files with 6 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp
View File

@ -1313,7 +1313,8 @@ static Instruction *foldFDivConstantDivisor(BinaryOperator &FDiv) {
return nullptr;
auto *RecipCFP = ConstantFP::get(FDiv.getType(), Reciprocal);
return BinaryOperator::CreateFMul(FDiv.getOperand(0), RecipCFP);
return BinaryOperator::CreateWithCopiedFlags(Instruction::FMul, RecipCFP,
FDiv.getOperand(0), &FDiv);
}
/// Try to reassociate C / X expressions where X includes another constant.
@ -1352,10 +1353,11 @@ Instruction *InstCombiner::visitFDiv(BinaryOperator &I) {
SQ.getWithInstruction(&I)))
return replaceInstUsesWith(I, V);
if (Instruction *FMul = foldFDivConstantDivisor(I)) {
FMul->copyFastMathFlags(&I);
if (Instruction *FMul = foldFDivConstantDivisor(I))
return FMul;
}
if (Instruction *NewFDiv = foldFDivConstantDividend(I))
return NewFDiv;
if (isa<Constant>(Op0))
if (SelectInst *SI = dyn_cast<SelectInst>(Op1))
@ -1394,9 +1396,6 @@ Instruction *InstCombiner::visitFDiv(BinaryOperator &I) {
return nullptr;
}
if (Instruction *NewFDiv = foldFDivConstantDividend(I))
return NewFDiv;
if (AllowReassociate) {
Value *X, *Y;
if (match(Op0, m_OneUse(m_FDiv(m_Value(X), m_Value(Y)))) &&