[RFC v2 01/38] add "this" pointers to more target APIs

Doug Evans dje@google.com
Sat Feb 8 05:19:00 GMT 2014


On Thu, Feb 6, 2014 at 12:54 PM, Tom Tromey <tromey@redhat.com> wrote:
> A subsequent pass introduces delegation helper functions to the target
> API.  This delegation is much cleaner if the target_ops pointer is
> directly available at delegation time.
>
> This patch adds the "this" pointer to various to_* methods for this
> purpose.
>
> This updates a number of ports which I am unable to test.  Please give
> them a look-over.  Any possible problem here is trivial, though, as
> all that is required is adding an argument to a function.

Hi.
I'm *not* suggesting wholesale changes (babysteps is totally fine with
me), but there's something I'm not sure about.

I think it was on IRC where there was a discussion of passing the
(pseudo-)vtable pointer as the self/this parameter being less
preferable to passing a more canonical self/this which in turn has the
(pseudo-)vtable pointer.  In this case we might not need such a change
until such time as we've already converted to C++ (and thus taking the
next step now could be time best spent elsewhere).  My question is to
ask for clarity on where you see the target API headed.  Will we need
a more canonical self/this pointer soon(-ish)?



More information about the Gdb-patches mailing list