This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [PATCH] Remove bogosities from dwarf2cfi.c:cfi_pop_frame()
- From: Richard Earnshaw <rearnsha at arm dot com>
- To: Andrew Cagney <ac131313 at ges dot redhat dot com>
- Cc: Mark Kettenis <kettenis at chello dot nl>, gdb-patches at sources dot redhat dot com, Richard dot Earnshaw at arm dot com
- Date: Wed, 10 Jul 2002 10:24:40 +0100
- Subject: Re: [PATCH] Remove bogosities from dwarf2cfi.c:cfi_pop_frame()
- Organization: ARM Ltd.
- Reply-to: Richard dot Earnshaw at arm dot com
> > Using alloca() here is pretty obvious. I discussed the other fix with
> > Michael Ludvig.
> >
> > Checked in.
> >
> > Mark
> >
> > Index: ChangeLog
> > from Mark Kettenis <kettenis@gnu.org>
> >
> > * dwarf2cfi.c (cfi_pop_frame): Use alloca() for regbuf.
> > Don't call get_current_frame().
> >
>
> Heads up,
>
> I think the write_register_bytes() call should be replaced by
> regcache_register_write(current_regcache, ). This is new code so it
> doesn't need to go through any of that nasty write_register_bytes() stuff.
>
> Andrew
>
>
Why is this code trying to poke directly into the regcache at all? AFAICT
it should be operating on the pseudo registers not the cache.
R.