[Lldb-commits] [PATCH] D97281: [lldb][AArch64] Add class for managing memory tags
Muhammad Omair Javaid via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Mon Jun 21 05:30:10 PDT 2021
omjavaid accepted this revision.
omjavaid added a comment.
This revision is now accepted and ready to land.
In D97281#2824678 <https://reviews.llvm.org/D97281#2824678>, @DavidSpickett wrote:
> @omjavaid As it stands, RemoveNonAddressBits here removes the top byte unconditionally.
>
> You asked for pointer signatures to be removed as well, which would be neatly solved by using the ABI plugin. Ok to land this as is, then write a follow up to migrate to using the ABI plugin?
Yes I think this is LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97281/new/
https://reviews.llvm.org/D97281
More information about the lldb-commits
mailing list