[PATCH] D44772: [DWARF] Simplify DWARFAddressRange::{intersects, contains}.

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

In https://reviews.llvm.org/D44772#1045463, @JDevlieghere wrote:

> That said, I'm not convinced we should consider fully-enclosed empty intervals as intersecting; the DWARF standard says that empty ranges can be ignored and therefore the verifier should not complain about this.


Okay, I'm convinced by that comment (I just checked, and it applies for .debug_ranges as well as .debug_rnglists). In more general terms, empty ranges probably should be considered for intersection, in my opinion, but not for these sort of ranges.


Repository:
  rL LLVM

https://reviews.llvm.org/D44772