This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Powerpc skip prologue
- From: Aleksandar Ristovski <aristovski at qnx dot com>
- To: gdb-patches at sources dot redhat dot com
- Date: Mon, 28 Jul 2008 16:58:24 -0400
- Subject: Powerpc skip prologue
Hello,
GDB assumes the gpr registers will be saved starting from a rN register up to r31. This assumption doesn't seem to be right. See this: http://sourceware.org/ml/gdb-patches/2007-12/msg00111.html
and this: http://sourceware.org/ml/gdb/2008-07/msg00279.html
So I devised a micro-patch for handling the saved gprs. I based it on the Daniel's observations and my own, by disassembling several functions - they appear to have prologue that will save several general purpose registers in the ascending register index order, but not up to r31. For example, r30 only, or r28,r29 etc.
Unfortunately, I can only test this on our (Neutrino) powerpc targets.
(No ChangeLog since I can not claim this is a final and correct solution in compliance with the ABI. If it turns out that ABI allows for saving registers non-sequentially or out-of order, e.g. r28, r30, r29, then this is not good and we need a more thorough patch that would allow for such situations).
Thanks,
Aleksandar Ristovski
QNX Software Systems
Index: gdb/rs6000-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/rs6000-tdep.c,v
retrieving revision 1.318
diff -u -p -r1.318 rs6000-tdep.c
--- gdb/rs6000-tdep.c 15 Jul 2008 18:32:06 -0000 1.318
+++ gdb/rs6000-tdep.c 28 Jul 2008 20:49:24 -0000
@@ -117,6 +117,7 @@ struct rs6000_framedata
by which we decrement sp to allocate
the frame */
int saved_gpr; /* smallest # of saved gpr */
+ int saved_gpr_max; /* Largest # of saved gpr */
int saved_fpr; /* smallest # of saved fpr */
int saved_vr; /* smallest # of saved vr */
int saved_ev; /* smallest # of saved ev */
@@ -1197,6 +1198,7 @@ skip_prologue (struct gdbarch *gdbarch,
memset (fdata, 0, sizeof (struct rs6000_framedata));
fdata->saved_gpr = -1;
+ fdata->saved_gpr_max = -1;
fdata->saved_fpr = -1;
fdata->saved_vr = -1;
fdata->saved_ev = -1;
@@ -1282,6 +1284,8 @@ skip_prologue (struct gdbarch *gdbarch,
op &= ~3UL;
fdata->gpr_offset = SIGNED_SHORT (op) + offset;
}
+ if (fdata->saved_gpr_max < reg)
+ fdata->saved_gpr_max = reg;
continue;
}
@@ -2571,7 +2575,7 @@ rs6000_frame_cache (struct frame_info *t
{
int i;
CORE_ADDR gpr_addr = cache->base + fdata.gpr_offset;
- for (i = fdata.saved_gpr; i < ppc_num_gprs; i++)
+ for (i = fdata.saved_gpr; i <= fdata.saved_gpr_max; i++)
{
cache->saved_regs[tdep->ppc_gp0_regnum + i].addr = gpr_addr;
gpr_addr += wordsize;