This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC] Document replacement for frame_register_read (deprecated).
- From: Tom Tromey <tromey at redhat dot com>
- To: Joel Brobecker <brobecker at adacore dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 23 Oct 2012 11:19:37 -0600
- Subject: Re: [RFC] Document replacement for frame_register_read (deprecated).
- References: <1350941919-9862-1-git-send-email-brobecker@adacore.com>
>>>>> "Joel" == Joel Brobecker <brobecker@adacore.com> writes:
Joel> This is inspired by a discussion from a long time ago:
Joel> http://www.sourceware.org/ml/gdb-patches/2011-03/msg01095.html
Joel> If you agree with the patch, we will also have to update the ARI
Joel> script to mention get_frame_register_value as the alternative.
Joel> And we might also want to rename the function as well. I don't think
Joel> we can rely on the ARI alone to avoid new uses of this function.
Joel> And once new uses are in, it's much harder to make sure we undo them.
Joel> Thoughts?
The patch seems reasonable to me.
Renaming seems like a decent idea, since there is a use in jit.c, which
was added long after this was deprecated.
I count 22 mentions (including the declaration) of frame_register_read
in the tree right now. Getting rid of all of them doesn't seem so
difficult, aside from the testing. This would be clearly better...
At least one of the current uses seems suspicious since it ignores the
result. I assume I'm missing some context though.
Tom