diff options
author | Mark Kettenis <kettenis@gnu.org> | 2004-11-29 19:52:44 +0000 |
---|---|---|
committer | Mark Kettenis <kettenis@gnu.org> | 2004-11-29 19:52:44 +0000 |
commit | 075ccec82f4bb6efd7aeb9420ba638578f0b3f59 (patch) | |
tree | 06c52d51f80a89cd3fdca0c6c6543807d46dd739 | |
parent | 9019af69c42e12d9c14823b7f9b28a73a5bb1c0f (diff) | |
download | gdb-075ccec82f4bb6efd7aeb9420ba638578f0b3f59.zip gdb-075ccec82f4bb6efd7aeb9420ba638578f0b3f59.tar.gz gdb-075ccec82f4bb6efd7aeb9420ba638578f0b3f59.tar.bz2 |
* sparc-tdep.c (X_RS1, X_SIMM13): New macros.
(sparc32_skip_prologue): Skip instructions that store arguments in
registers into their corresponding stack slots.
-rw-r--r-- | gdb/ChangeLog | 4 | ||||
-rw-r--r-- | gdb/sparc-tdep.c | 33 |
2 files changed, 36 insertions, 1 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 736213d..67c81d8 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,9 @@ 2004-11-29 Mark Kettenis <kettenis@gnu.org> + * sparc-tdep.c (X_RS1, X_SIMM13): New macros. + (sparc32_skip_prologue): Skip instructions that store arguments in + registers into their corresponding stack slots. + * config/m68k/nbsdaout.mh (NAT_FILE): Set to solib.h instead of tm-solib.h. * config/m68k/obsd.mh (NAT_FILE): Likewise. diff --git a/gdb/sparc-tdep.c b/gdb/sparc-tdep.c index 9facbad..b4051c9 100644 --- a/gdb/sparc-tdep.c +++ b/gdb/sparc-tdep.c @@ -80,10 +80,12 @@ struct regset; #define X_OP2(i) (((i) >> 22) & 0x7) #define X_IMM22(i) ((i) & 0x3fffff) #define X_OP3(i) (((i) >> 19) & 0x3f) +#define X_RS1(i) (((i) >> 14) & 0x1f) #define X_I(i) (((i) >> 13) & 1) /* Sign extension macros. */ #define X_DISP22(i) ((X_IMM22 (i) ^ 0x200000) - 0x200000) #define X_DISP19(i) ((((i) & 0x7ffff) ^ 0x40000) - 0x40000) +#define X_SIMM13(i) ((((i) & 0x1fff) ^ 0x1000) - 0x1000) /* Fetch the instruction at PC. Instructions are always big-endian even if the processor operates in little-endian mode. */ @@ -609,7 +611,36 @@ sparc32_skip_prologue (CORE_ADDR start_pc) return sal.end; } - return sparc_analyze_prologue (start_pc, 0xffffffffUL, &cache); + start_pc = sparc_analyze_prologue (start_pc, 0xffffffffUL, &cache); + + /* The psABI says that "Although the first 6 words of arguments + reside in registers, the standard stack frame reserves space for + them.". It also suggests that a function may use that space to + "write incoming arguments 0 to 5" into that space, and that's + indeed what GCC seems to be doing. In that case GCC will + generate debug information that points to the stack slots instead + of the registers, so we should consider the instructions that + write out these incoming arguments onto the stack. Of course we + only need to do this if we have a stack frame. */ + + while (!cache.frameless_p) + { + unsigned long insn = sparc_fetch_instruction (start_pc); + + /* Recognize instructions that store incoming arguments in + %i0...%i5 into the corresponding stack slot. */ + if (X_OP (insn) == 3 && (X_OP3 (insn) & 0x3c) == 0x04 && X_I (insn) + && (X_RD (insn) >= 24 && X_RD (insn) <= 29) && X_RS1 (insn) == 30 + && X_SIMM13 (insn) == 68 + (X_RD (insn) - 24) * 4) + { + start_pc += 4; + continue; + } + + break; + } + + return start_pc; } /* Normal frames. */ |