This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
[RFA] Fix PR tdep/1291, SH prologue scanning bug
- From: Fred Fish <fnf at ninemoons dot com>
- To: gdb-patches at sources dot redhat dot com
- Cc: fnf at redhat dot com
- Date: Thu, 19 Feb 2004 23:44:38 -0700
- Subject: [RFA] Fix PR tdep/1291, SH prologue scanning bug
- Reply-to: fnf at redhat dot com
This patch fixes the bug reported in PR 1291. It is based on the suggested
patch included in the PR. I believe it is small enough to not need a
copyright assignment, but recent events may have changed that. :-(
-Fred
2004-02-19 Fred Fish <fnf@redhat.com>
Fix for PR tdep/1291 as suggested by inaba@src.ricoh.co.jp
* sh-tdep.c (IS_MOV_R3): Rename to IS_MOV_IMM_R3 and fix pattern.
(IS_ADD_R3SP): Rename to IS_ADD_R3_SP for consistency.
(IS_MOVW_R1): New macro.
(IS_MOVL_R1): New macro.
(IS_SUB_R1_SP): New macro.
(sh_analyze_prologue): Add r1_val local var and initialize to zero.
Use IS_MOVW_R1, IS_MOVL_R1, and IS_SUB_R1_SP to recognize use of
stack allocation via constant loaded into r1.
Index: sh-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/sh-tdep.c,v
retrieving revision 1.165
diff -c -p -r1.165 sh-tdep.c
*** sh-tdep.c 20 Feb 2004 00:16:16 -0000 1.165
--- sh-tdep.c 20 Feb 2004 06:32:56 -0000
*************** sh_breakpoint_from_pc (CORE_ADDR *pcptr,
*** 330,341 ****
r15+imm-->r15 */
#define IS_ADD_IMM_SP(x) (((x) & 0xff00) == 0x7f00)
! #define IS_MOV_R3(x) (((x) & 0xff00) == 0x1a00)
#define IS_SHLL_R3(x) ((x) == 0x4300)
/* ADD r3,r15 0011111100111100
r15+r3-->r15 */
! #define IS_ADD_R3SP(x) ((x) == 0x3f3c)
/* FMOV.S FRm,@-Rn Rn-4-->Rn, FRm-->(Rn) 1111nnnnmmmm1011
FMOV DRm,@-Rn Rn-8-->Rn, DRm-->(Rn) 1111nnnnmmm01011
--- 330,355 ----
r15+imm-->r15 */
#define IS_ADD_IMM_SP(x) (((x) & 0xff00) == 0x7f00)
! /* MOV #imm,r3 11100011iiiiiiii
! imm-->r3 */
! #define IS_MOV_IMM_R3(x) (((x) & 0xff00) == 0xe300)
#define IS_SHLL_R3(x) ((x) == 0x4300)
/* ADD r3,r15 0011111100111100
r15+r3-->r15 */
! #define IS_ADD_R3_SP(x) ((x) == 0x3f3c)
!
! /* MOV.W @(disp, pc), r1 10010001dddddddd
! (disp * 2 + pc + 4)-->r1 */
! #define IS_MOVW_R1(x) (((x) & 0xff00) == 0x9100)
!
! /* MOV.L @(disp, pc), r1 11010001dddddddd
! (disp * 4 + pc + 4)-->r1 */
! #define IS_MOVL_R1(x) (((x) & 0xff00) == 0xd100)
!
! /* SUB r1,r15 00111111100011000
! r15-r1-->r15 */
! #define IS_SUB_R1_SP(x) ((x) == 0x3f18)
/* FMOV.S FRm,@-Rn Rn-4-->Rn, FRm-->(Rn) 1111nnnnmmmm1011
FMOV DRm,@-Rn Rn-8-->Rn, DRm-->(Rn) 1111nnnnmmm01011
*************** sh_analyze_prologue (CORE_ADDR pc, CORE_
*** 394,399 ****
--- 408,414 ----
CORE_ADDR opc;
int offset;
int sav_offset = 0;
+ int r1_val = 0;
int r3_val = 0;
int reg, sav_reg = -1;
*************** sh_analyze_prologue (CORE_ADDR pc, CORE_
*** 415,421 ****
cache->saved_regs[PR_REGNUM] = cache->sp_offset;
cache->sp_offset += 4;
}
! else if (IS_MOV_R3 (inst))
{
r3_val = ((inst & 0xff) ^ 0x80) - 0x80;
}
--- 430,436 ----
cache->saved_regs[PR_REGNUM] = cache->sp_offset;
cache->sp_offset += 4;
}
! else if (IS_MOV_IMM_R3 (inst))
{
r3_val = ((inst & 0xff) ^ 0x80) - 0x80;
}
*************** sh_analyze_prologue (CORE_ADDR pc, CORE_
*** 423,431 ****
{
r3_val <<= 1;
}
! else if (IS_ADD_R3SP (inst))
{
cache->sp_offset += -r3_val;
}
else if (IS_ADD_IMM_SP (inst))
{
--- 438,458 ----
{
r3_val <<= 1;
}
! else if (IS_ADD_R3_SP (inst))
{
cache->sp_offset += -r3_val;
+ }
+ else if (IS_MOVW_R1 (inst))
+ {
+ r1_val = read_memory_integer (pc + 4 + (inst & 0xff) * 2, 2);
+ }
+ else if (IS_MOVL_R1 (inst))
+ {
+ r1_val = read_memory_integer (pc + 4 + (inst & 0xff) * 4, 4);
+ }
+ else if (IS_SUB_R1_SP (inst))
+ {
+ cache->sp_offset += r1_val;
}
else if (IS_ADD_IMM_SP (inst))
{