diff options
author | Richard Earnshaw <richard.earnshaw@arm.com> | 2010-03-31 22:10:07 +0000 |
---|---|---|
committer | Richard Earnshaw <richard.earnshaw@arm.com> | 2010-03-31 22:10:07 +0000 |
commit | d476da0ef37ba5ca612a536c69bfe2b859e17c46 (patch) | |
tree | 16612e185ffd3a08045cab3ecbbc6c3a52e94cc3 | |
parent | 77aabdd2d99a9f2171d284147fe8be83e3da3243 (diff) | |
download | gdb-d476da0ef37ba5ca612a536c69bfe2b859e17c46.zip gdb-d476da0ef37ba5ca612a536c69bfe2b859e17c46.tar.gz gdb-d476da0ef37ba5ca612a536c69bfe2b859e17c46.tar.bz2 |
* arm-tdep.c (thumb_get_next_pc_raw): Correctly detect TBH
instructions. Use the PC+4 if the base of the TBB or TBH is the
PC register.
-rw-r--r-- | gdb/ChangeLog | 9 | ||||
-rw-r--r-- | gdb/arm-tdep.c | 20 |
2 files changed, 23 insertions, 6 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index a83755e..58b54dd 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2010-03-31 Richard Earnshaw <rearnsha@arm.com> + + * arm-tdep.c (thumb_get_next_pc_raw): Correctly detect TBH + instructions. Use the PC+4 if the base of the TBB or TBH is the + PC register. + 2010-03-31 Jan Kratochvil <jan.kratochvil@redhat.com> Fix crash on reading wrong function declaration DWARF. @@ -14,7 +20,8 @@ 2010-03-31 Pierre Muller <muller@ics.u-strasbg.fr> - * src/gdb/remote.c (end_thread): ARI fix: Use xstrdup instead of strdup. + * src/gdb/remote.c (end_thread): ARI fix: Use xstrdup instead of + strdup. 2010-03-30 Keith Seitz <keiths@redhat.com> diff --git a/gdb/arm-tdep.c b/gdb/arm-tdep.c index f6a8436..d4248f9 100644 --- a/gdb/arm-tdep.c +++ b/gdb/arm-tdep.c @@ -2959,19 +2959,29 @@ thumb_get_next_pc_raw (struct frame_info *frame, CORE_ADDR pc, int insert_bkpt) else if ((inst1 & 0xfff0) == 0xe8d0 && (inst2 & 0xfff0) == 0xf000) { /* TBB. */ - CORE_ADDR table, offset, length; + CORE_ADDR tbl_reg, table, offset, length; + + tbl_reg = bits (inst1, 0, 3); + if (tbl_reg == 0x0f) + table = pc + 4; /* Regcache copy of PC isn't right yet. */ + else + table = get_frame_register_unsigned (frame, tbl_reg); - table = get_frame_register_unsigned (frame, bits (inst1, 0, 3)); offset = get_frame_register_unsigned (frame, bits (inst2, 0, 3)); length = 2 * get_frame_memory_unsigned (frame, table + offset, 1); nextpc = pc_val + length; } - else if ((inst1 & 0xfff0) == 0xe8d0 && (inst2 & 0xfff0) == 0xf000) + else if ((inst1 & 0xfff0) == 0xe8d0 && (inst2 & 0xfff0) == 0xf010) { /* TBH. */ - CORE_ADDR table, offset, length; + CORE_ADDR tbl_reg, table, offset, length; + + tbl_reg = bits (inst1, 0, 3); + if (tbl_reg == 0x0f) + table = pc + 4; /* Regcache copy of PC isn't right yet. */ + else + table = get_frame_register_unsigned (frame, tbl_reg); - table = get_frame_register_unsigned (frame, bits (inst1, 0, 3)); offset = 2 * get_frame_register_unsigned (frame, bits (inst2, 0, 3)); length = 2 * get_frame_memory_unsigned (frame, table + offset, 2); nextpc = pc_val + length; |