]> granicus.if.org Git - llvm/commitdiff
Revert "[DWARF] Delete redundant check in DWARFAddressRange::intersects"
authorFangrui Song <maskray@google.com>
Mon, 15 Apr 2019 09:01:10 +0000 (09:01 +0000)
committerFangrui Song <maskray@google.com>
Mon, 15 Apr 2019 09:01:10 +0000 (09:01 +0000)
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

include/llvm/DebugInfo/DWARF/DWARFAddressRange.h

index 128b410d79e4d673ea21882891069d215d51c598..56d46c696fd6e2d8157c21b61f0c805de5df0912 100644 (file)
@@ -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;
   }