[PATCH] mips: set significant_addr_bit to number of bits in CORE_ADDR
Vlad Ivanov
vlad.ivanov@lab-systems.ru
Thu Mar 15 14:39:00 GMT 2018
MIPS targets use signed PC values. Since commit a0de8c21
single-stepping on these targets didn't work due to the addition of
address_significant in adjust_breakpoint_address - sign bits were cut
off because of default value of significant_addr_bit. With this
commit, significant_addr_bit in gdbarch is set to the number of bits
in CORE_ADDR so breakpoint address comparison works again for MIPS
targets.
* mips-tdep.c (mips_gdbarch_init): Set significant_addr_bit
---
gdb/mips-tdep.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c
index f9f84c4d48..5d38168af0 100644
--- a/gdb/mips-tdep.c
+++ b/gdb/mips-tdep.c
@@ -8685,6 +8685,7 @@ mips_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches)
/* Add/remove bits from an address. The MIPS needs be careful to
ensure that all 32 bit addresses are sign extended to 64 bits. */
set_gdbarch_addr_bits_remove (gdbarch, mips_addr_bits_remove);
+ set_gdbarch_significant_addr_bit (gdbarch, sizeof (CORE_ADDR) * HOST_CHAR_BIT);
/* Unwind the frame. */
set_gdbarch_unwind_pc (gdbarch, mips_unwind_pc);
--
2.14.3
More information about the Gdb-patches
mailing list