labath wrote:

There isn't a good reason for using GetAddressRange, and removing it is what 
I'm trying to do. But rather than trying to do it all at once, I'm doing it in 
pieces as some of the changes (particularly those that need to iterate over a 
set of ranges) are not trivial. This way I can also verify that each change 
does what it's supposed to do.

I changed most of the trivial call sites when I introduced GetAddress (in 
#115836). In retrospect, this is probably trivial enough that it could have 
been a part of that, but I skipped it then because the mapping part looked 
scary.

https://github.com/llvm/llvm-project/pull/124938
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to