diff options
author | Fangrui Song <maskray@google.com> | 2018-03-27 19:05:02 +0000 |
---|---|---|
committer | Fangrui Song <maskray@google.com> | 2018-03-27 19:05:02 +0000 |
commit | fc5dabe738816379d749d8df93b5907b04982cb4 (patch) | |
tree | 21760b4e89469a8a7b9762b943641c447fdaba69 | |
parent | adf5a32ec57addb326ff6cd13f1e9e0a6a10bdb8 (diff) | |
download | bcm5719-llvm-fc5dabe738816379d749d8df93b5907b04982cb4.tar.gz bcm5719-llvm-fc5dabe738816379d749d8df93b5907b04982cb4.zip |
[DWARF] Simplify DWARFAddressRange::contains
This transform is valid because the ranges have been validated (LowPC <= HighPC).
Differential Revision: https://reviews.llvm.org/D44772
llvm-svn: 328655
-rw-r--r-- | llvm/include/llvm/DebugInfo/DWARF/DWARFAddressRange.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/llvm/include/llvm/DebugInfo/DWARF/DWARFAddressRange.h b/llvm/include/llvm/DebugInfo/DWARF/DWARFAddressRange.h index c7778897ec6..5a7df5c353e 100644 --- a/llvm/include/llvm/DebugInfo/DWARF/DWARFAddressRange.h +++ b/llvm/include/llvm/DebugInfo/DWARF/DWARFAddressRange.h @@ -36,17 +36,17 @@ struct DWARFAddressRange { /// Returns true if [LowPC, HighPC) intersects with [RHS.LowPC, RHS.HighPC). bool intersects(const DWARFAddressRange &RHS) const { + assert(valid() && RHS.valid()); // Empty ranges can't intersect. if (LowPC == HighPC || RHS.LowPC == RHS.HighPC) return false; - return (LowPC < RHS.HighPC) && (HighPC > RHS.LowPC); + return LowPC < RHS.HighPC && RHS.LowPC < HighPC; } /// Returns true if [LowPC, HighPC) fully contains [RHS.LowPC, RHS.HighPC). bool contains(const DWARFAddressRange &RHS) const { - if (LowPC <= RHS.LowPC && RHS.LowPC <= HighPC) - return LowPC <= RHS.HighPC && RHS.HighPC <= HighPC; - return false; + assert(valid() && RHS.valid()); + return LowPC <= RHS.LowPC && RHS.HighPC <= HighPC; } void dump(raw_ostream &OS, uint32_t AddressSize, |