diff options
author | Omair Javaid <omair.javaid@linaro.org> | 2018-05-26 05:58:51 +0500 |
---|---|---|
committer | Omair Javaid <omair.javaid@linaro.org> | 2018-05-31 08:26:16 +0500 |
commit | 8679931c2c9f97ad182f8f0a9e8fab20305719b5 (patch) | |
tree | 0d6c9a2d405bfd757ead0c386e861ab286505498 /gdb/abug-rom.c | |
parent | fcfc3b91251aa790d9f92eb97359797693fe6efb (diff) | |
download | binutils-8679931c2c9f97ad182f8f0a9e8fab20305719b5.zip binutils-8679931c2c9f97ad182f8f0a9e8fab20305719b5.tar.gz binutils-8679931c2c9f97ad182f8f0a9e8fab20305719b5.tar.bz2 |
Unset gdbarch significant_addr_bit by default
This patch fixes a bug introduced by fix to AArch64 pointer tagging.
In our fix for tagged pointer support our agreed approach was to sign
extend user-space address after clearing tag bits. This is not same
for all architectures and this patch allows sign extension for
addresses on targets which specifically set significant_addr_bit.
More information about patch that caused the issues and discussion
around tagged pointer support can be found in links below:
https://sourceware.org/ml/gdb-patches/2018-05/msg00000.html
https://sourceware.org/ml/gdb-patches/2017-12/msg00159.html
gdb/ChangeLog:
2018-05-31 Omair Javaid <omair.javaid@linaro.org>
PR gdb/23210
* gdbarch.sh (significant_addr_bit): Default to zero when
not set by target architecture.
* gdbarch.c: Re-generated.
* utils.c (address_significant): Update.
Diffstat (limited to 'gdb/abug-rom.c')
0 files changed, 0 insertions, 0 deletions