diff options
| author | shkzhang <shkzhang@cn.ibm.com> | 2019-12-11 04:46:00 -0500 |
|---|---|---|
| committer | shkzhang <shkzhang@cn.ibm.com> | 2019-12-11 04:46:00 -0500 |
| commit | 1408e7e17525287c596a8f575957aecb684fa75d (patch) | |
| tree | 9dbf83122387cd937f9ece7a4383d8139a1847a2 /llvm/tools/llvm-readobj/llvm-readobj.cpp | |
| parent | 11f311875f092e59cac2936b54f922b968e615e3 (diff) | |
| download | llvm-1408e7e17525287c596a8f575957aecb684fa75d.zip llvm-1408e7e17525287c596a8f575957aecb684fa75d.tar.gz llvm-1408e7e17525287c596a8f575957aecb684fa75d.tar.bz2 | |
[PowerPC] [CodeGen] Use MachineBranchProbabilityInfo in EarlyIfPredicator to avoid the potential bug
Summary:
In the function `EarlyIfPredicator::shouldConvertIf()`, we call
`TII->isProfitableToIfCvt()` with `BranchProbability::getUnknown()`, it may
cause the potential assertion error for those hook which use `BranchProbability`
in `isProfitableToIfCvt()`, for example `SystemZ`.
`SystemZ` use `Probability < BranchProbability(1, 8))` in the function
`SystemZInstrInfo::isProfitableToIfCvt()`, if we call this function with
`BranchProbability::getUnknown()`, it will cause assertion error.
This patch is to fix the potential bug.
Reviewed By: ThomasRaoux
Differential Revision: https://reviews.llvm.org/D71273
Diffstat (limited to 'llvm/tools/llvm-readobj/llvm-readobj.cpp')
0 files changed, 0 insertions, 0 deletions