RPCSX/llvm
1
0
Fork 0
mirror of https://github.com/RPCSX/llvm.git synced 2025-02-27 16:38:30 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

De-duplicate the two implementations of ARMBaseInstrInfo::isProfitableToIfCvt() [NFC]

Browse Source 
Reviewers: congh, rengolin

Subscribers: aemerson, llvm-commits

Differential Revision: https://reviews.llvm.org/D30934

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@297738 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Artyom Skrobov 2017-03-14 13:38:45 +00:00
parent 1a739b3383
commit 0117588515
1 changed files with 5 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
lib/Target/ARM/ARMBaseInstrInfo.cpp
View File

@ -1789,25 +1789,17 @@ isProfitableToIfCvt(MachineBasicBlock &MBB,
}
}
}
// Attempt to estimate the relative costs of predication versus branching.
// Here we scale up each component of UnpredCost to avoid precision issue when
// scaling NumCycles by Probability.
const unsigned ScalingUpFactor = 1024;
unsigned UnpredCost = Probability.scale(NumCycles * ScalingUpFactor);
UnpredCost += ScalingUpFactor; // The branch itself
UnpredCost += Subtarget.getMispredictionPenalty() * ScalingUpFactor / 10;
return (NumCycles + ExtraPredCycles) * ScalingUpFactor <= UnpredCost;
return isProfitableToIfCvt(MBB, NumCycles, ExtraPredCycles,
MBB, 0, 0, Probability);
}
bool ARMBaseInstrInfo::
isProfitableToIfCvt(MachineBasicBlock &TMBB,
isProfitableToIfCvt(MachineBasicBlock &,
unsigned TCycles, unsigned TExtra,
MachineBasicBlock &FMBB,
MachineBasicBlock &,
unsigned FCycles, unsigned FExtra,
BranchProbability Probability) const {
if (!TCycles || !FCycles)
if (!TCycles)
return false;
// Attempt to estimate the relative costs of predication versus branching.