Re: [RFA] mips_init_extra_frame_info: Set saved_regs value for SP_REGNUM

[Kevin Buettner <kevinb@redhat.com>]

On Aug 9, 12:21am, Andrew Cagney wrote:

> > 	* mips-tdep.c (mips_init_extra_frame_info): Initialize SP_REGNUM's
> > 	saved regs value.
> > 	
> > Index: mips-tdep.c
> > ===================================================================
> > RCS file: /cvs/src/src/gdb/mips-tdep.c,v
> > retrieving revision 1.93
> > diff -u -p -r1.93 mips-tdep.c
> > --- mips-tdep.c	8 Aug 2002 23:32:52 -0000	1.93
> > +++ mips-tdep.c	9 Aug 2002 00:13:05 -0000
> > @@ -2342,6 +2342,7 @@ mips_init_extra_frame_info (int fromleaf
> >  	      memcpy (fci->saved_regs, temp_saved_regs, SIZEOF_FRAME_SAVED_REGS);
> >  	      fci->saved_regs[PC_REGNUM]
> >  		= fci->saved_regs[RA_REGNUM];
> > +	      fci->saved_regs[SP_REGNUM] = fci->frame;
> >  	    }
> >  	}
> 
> Yes, ok.  Just add a comment reminding the reader that this is setting 
> the SP for the previous frame.

Committed.  For the record, here's what went in:

Index: mips-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/mips-tdep.c,v
retrieving revision 1.94
diff -u -p -r1.94 mips-tdep.c
--- mips-tdep.c	9 Aug 2002 01:42:41 -0000	1.94
+++ mips-tdep.c	9 Aug 2002 20:28:30 -0000
@@ -2342,6 +2351,12 @@ mips_init_extra_frame_info (int fromleaf
 	      memcpy (fci->saved_regs, temp_saved_regs, SIZEOF_FRAME_SAVED_REGS);
 	      fci->saved_regs[PC_REGNUM]
 		= fci->saved_regs[RA_REGNUM];
+	      /* Set value of previous frame's stack pointer.  Remember that
+	         saved_regs[SP_REGNUM] is special in that it contains the
+		 value of the stack pointer register.  The other saved_regs
+		 values are addresses (in the inferior) at which a given
+		 register's value may be found.  */
+	      fci->saved_regs[SP_REGNUM] = fci->frame;
 	    }
 	}