This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: New ARI warning Sat Mar 19 01:54:11 UTC 2011
- From: "Ulrich Weigand" <uweigand at de dot ibm dot com>
- To: brobecker at adacore dot com (Joel Brobecker)
- Cc: pedro at codesourcery dot com (Pedro Alves), gdb-patches at sourceware dot org
- Date: Fri, 25 Mar 2011 17:25:15 +0100 (CET)
- Subject: Re: New ARI warning Sat Mar 19 01:54:11 UTC 2011
Joel Brobecker wrote:
> > I think that all users that require this additional status information
> > should just use the (new) get_frame_register_value, and look at that
> > value's properties.
> >
> > So I do think that frame_register_read ought to stay deprecated;
> > we need to remove those extraneous frame register routines ...
>
> Pedro, if you agree, I'll add a comment to that effect next to
> frame_register_read. Alternatively, we could try to reimplement
> frame_register_read using get_frame_register_value, although
> I get the impression that we wanted to start getting away from
> reading into buffers.
Oh, frame_register_read is already implemented in terms of
get_frame_register_value (well, frame_unwind_register_value
really, but that's the same thing).
I agree that the goal should be to push that implementation up
to the callers, and get rid of the buffer-based accessors.
Bye,
Ulrich
--
Dr. Ulrich Weigand
GNU Toolchain for Linux on System z and Cell BE
Ulrich.Weigand@de.ibm.com