[PATCH] D44772: [DWARF] Simplify DWARFAddressRange::contains

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 27 02:54:46 PDT 2018


JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D44772#1048988, @jhenderson wrote:

> LGTM. Any reason you decided not to change `intersects()`?


Because we concluded that would not conform to the dwarf standard, right?

> @JDevlieghere - any more comments from you?

The current diff LGTM.


Repository:
  rL LLVM

https://reviews.llvm.org/D44772





More information about the llvm-commits mailing list