This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [rfc]: Replace current_gdbarch in sentinel-frame.c
- From: Michael Snyder <msnyder at specifix dot com>
- To: Markus Deuling <deuling at de dot ibm dot com>
- Cc: GDB Patches <gdb-patches at sourceware dot org>, Ulrich Weigand <uweigand at de dot ibm dot com>
- Date: Fri, 15 Feb 2008 09:51:12 -0800
- Subject: Re: [rfc]: Replace current_gdbarch in sentinel-frame.c
- References: <47B55A49.2040107@de.ibm.com> <1203097447.19253.128.camel@localhost.localdomain>
On Fri, 2008-02-15 at 09:44 -0800, Michael Snyder wrote:
> On Fri, 2008-02-15 at 10:24 +0100, Markus Deuling wrote:
> > Hi,
> >
> > this is a trivial replacement of current_gdbarch in sentinel-frame.c. Tested on x86.
> >
> > Ok to commit?
>
> Looks fine to me.
And it seems I was premature.
I defer to Ulrich. ;-)
>
> >
> >
> > ChangeLog:
> >
> > * sentinel-frame.c (sentinel_frame_prev_register): Use get_frame_arch
> > to get at the current architecture.
> >
> > Regards,
> > Markus
> >
> >
> > plain text document attachment (diff-sentinel)
> > diff -urpN src/gdb/sentinel-frame.c dev/gdb/sentinel-frame.c
> > --- src/gdb/sentinel-frame.c 2008-01-01 23:53:12.000000000 +0100
> > +++ dev/gdb/sentinel-frame.c 2008-02-15 10:07:36.000000000 +0100
> > @@ -54,7 +54,7 @@ sentinel_frame_prev_register (struct fra
> > onto the corresponding hardware register. */
> > *optimized = 0;
> > *lvalp = lval_register;
> > - *addrp = register_offset_hack (current_gdbarch, regnum);
> > + *addrp = register_offset_hack (get_frame_arch (next_frame), regnum);
> > *realnum = regnum;
> >
> > /* If needed, find and return the value of the register. */
>