[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