[PATCH v2][PR gdb/23210] Unset gdbarch significant_addr_bit by default
Pedro Alves
palves@redhat.com
Tue May 29 13:12:00 GMT 2018
On 05/26/2018 01:58 AM, Omair Javaid wrote:
> This version fixes the typo and gdbarch.sh issue highlighted in review.
>
> LGTM?
I think the resulting API ends up a little weird. Maybe a better
fit would be something around:
CORE_ADDR gdbarch_canonical_address (gdbarch *, CORE_ADDR addr);
with the default implementation being a nop [return ADDR] while
Aarch64's implementation would clear the tag and sign extend.
Dunno, not sure, I really did not think that through, and, as
I mentioned before, I'm totally OK with this as is, we understand
the issues and can always change it later.
More important right now is unbreaking 32-bit x86 and the branch.
Thus, OK as is. Please push to master and branch.
Thanks,
Pedro Alves
More information about the Gdb-patches
mailing list