[PATCH] D44772: [DWARF] Simplify DWARFAddressRange::{intersects, contains}.
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 22 00:28:15 PDT 2018
MaskRay created this revision.
MaskRay added a reviewer: jhenderson.
Herald added subscribers: llvm-commits, JDevlieghere.
This transform is valid because the ranges have been validated (LowPC <= HighPC).
Repository:
rL LLVM
https://reviews.llvm.org/D44772
Files:
include/llvm/DebugInfo/DWARF/DWARFAddressRange.h
Index: include/llvm/DebugInfo/DWARF/DWARFAddressRange.h
===================================================================
--- include/llvm/DebugInfo/DWARF/DWARFAddressRange.h
+++ include/llvm/DebugInfo/DWARF/DWARFAddressRange.h
@@ -36,17 +36,12 @@
/// Returns true if [LowPC, HighPC) intersects with [RHS.LowPC, RHS.HighPC).
bool intersects(const DWARFAddressRange &RHS) const {
- // 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;
+ return LowPC <= RHS.LowPC && RHS.HighPC <= HighPC;
}
void dump(raw_ostream &OS, uint32_t AddressSize,
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44772.139419.patch
Type: text/x-patch
Size: 999 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180322/3116a15e/attachment.bin>
More information about the llvm-commits
mailing list