From: Fangrui Song Date: Mon, 15 Apr 2019 09:01:10 +0000 (+0000) Subject: Revert "[DWARF] Delete redundant check in DWARFAddressRange::intersects" X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=460838586d8b9581924d67161a425de6ba629ba8;p=llvm Revert "[DWARF] Delete redundant check in DWARFAddressRange::intersects" This reverts rL358377. It isn't redundant, though the semantic seems weird. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@358381 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/llvm/DebugInfo/DWARF/DWARFAddressRange.h b/include/llvm/DebugInfo/DWARF/DWARFAddressRange.h index 128b410d79e..56d46c696fd 100644 --- a/include/llvm/DebugInfo/DWARF/DWARFAddressRange.h +++ b/include/llvm/DebugInfo/DWARF/DWARFAddressRange.h @@ -36,6 +36,9 @@ 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 && RHS.LowPC < HighPC; }