diff options
author | Hans Wennborg <hans@hanshq.net> | 2015-12-01 03:49:42 +0000 |
---|---|---|
committer | Hans Wennborg <hans@hanshq.net> | 2015-12-01 03:49:42 +0000 |
commit | 8e83fe2e97c232086674f9f04b00043bccb89629 (patch) | |
tree | 0d85b9ef0b9497932253d4220d77b5222c6de0c9 /lib/Support/BranchProbability.cpp | |
parent | 5d2885f554db8eb89a3e2b376c4d6507d456eed2 (diff) |
Revert r254348: "Replace all weight-based interfaces in MBB with probability-based interfaces, and update all uses of old interfaces."
and the follow-up r254356: "Fix a bug in MachineBlockPlacement that may cause assertion failure during BranchProbability construction."
Asserts were firing in Chromium builds. See PR25687.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@254366 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Support/BranchProbability.cpp')
-rw-r--r-- | lib/Support/BranchProbability.cpp | 20 |
1 files changed, 2 insertions, 18 deletions
diff --git a/lib/Support/BranchProbability.cpp b/lib/Support/BranchProbability.cpp index 771d02c0aa3..3b0f6e6f06e 100644 --- a/lib/Support/BranchProbability.cpp +++ b/lib/Support/BranchProbability.cpp @@ -22,14 +22,11 @@ using namespace llvm; const uint32_t BranchProbability::D; raw_ostream &BranchProbability::print(raw_ostream &OS) const { - if (isUnknown()) - return OS << "?%"; - // Get a percentage rounded to two decimal digits. This avoids // implementation-defined rounding inside printf. double Percent = rint(((double)N / D) * 100.0 * 100.0) / 100.0; - return OS << format("0x%08" PRIx32 " / 0x%08" PRIx32 " = %.2f%%", N, D, - Percent); + OS << format("0x%08" PRIx32 " / 0x%08" PRIx32 " = %.2f%%", N, D, Percent); + return OS; } void BranchProbability::dump() const { print(dbgs()) << '\n'; } @@ -46,19 +43,6 @@ BranchProbability::BranchProbability(uint32_t Numerator, uint32_t Denominator) { } } -BranchProbability -BranchProbability::getBranchProbability(uint64_t Numerator, - uint64_t Denominator) { - assert(Numerator <= Denominator && "Probability cannot be bigger than 1!"); - // Scale down Denominator to fit in a 32-bit integer. - int Scale = 0; - while (Denominator > UINT32_MAX) { - Denominator >>= 1; - Scale++; - } - return BranchProbability(Numerator >> Scale, Denominator); -} - // If ConstD is not zero, then replace D by ConstD so that division and modulo // operations by D can be optimized, in case this function is not inlined by the // compiler. |