[Lldb-commits] [PATCH] D102757: [lldb] Remove non address bits when looking up memory regions

David Spickett via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed May 26 04:09:56 PDT 2021


DavidSpickett added a comment.

In D102757#2781561 <https://reviews.llvm.org/D102757#2781561>, @omjavaid wrote:

> Now that we are stripping away top byte is there any information that may be useful for the remote side and we are removing that on the host side. I am thinking why we should strip top byte on host side rather than making it the responsibility of the remote end?

I don't think there'd be anything in there the remote needs. Not for `qMemoryRegionInfo`.

lldb-server will need to remove memory tags somehow anyway (because the tag packet spec says you can send tagged addresses) so we could have it do all of it. My unknown there is "non remote" kinda targets like ELF cores or minidumps, does using the ABI for just those get messy.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102757/new/

https://reviews.llvm.org/D102757



More information about the lldb-commits mailing list