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

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