Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Kevin Buettner <kevinb@redhat.com>
To: Andrew Cagney <ac131313@redhat.com>,
	gdb-patches@sources.redhat.com,
	Kevin Buettner <kevinb@redhat.com>,
	Jason R Thorpe <thorpej@wasabisystems.com>
Subject: Re: [rfa?] Implement ppc32 SYSV {extract,store} return value
Date: Mon, 22 Sep 2003 21:58:00 -0000	[thread overview]
Message-ID: <1030922215845.ZM29725@localhost.localdomain> (raw)
In-Reply-To: Andrew Cagney <ac131313@redhat.com> "[rfa?] Implement ppc32 SYSV {extract,store} return value" (Sep 17,  5:54pm)

On Sep 17,  5:54pm, Andrew Cagney wrote:

> If nothing else I'd really like a comment on the general approach taken 
> - the netbsd function wrapping a more generic method,

What are you referring to here?  I looked at the netbsd portion of
your patch and I don't see what you're referring to.  I do see
that your using the ppc_sys_v_abi_... version for extract_return_value
and store_return_value.  But I think that's okay until we find that
it's broken in ways not already handled by the code you've just added.

(This is different than the case that I commented on previously -- In
which it appeared that code for one ABI was reusable by another.  In
such a case, I think clearly distinguishing the code implementing the
two ABIs is a good idea.  For NetBSD, it's supposed to be implementing
the System V ABI, right?  That being the case, I think it's just fine
to do things the way you've arranged it.)

> and a better word than "broken" I guess.

At the moment, I can't think of anything better.  (Or, to put it another
way, anything that I've thought of ends up being overly verbose.)

Anyway, your patch looks okay to me.  Feel free to check it in.

Thanks,

Kevin


  reply	other threads:[~2003-09-22 21:58 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-09-17 21:54 Andrew Cagney
2003-09-22 21:58 ` Kevin Buettner [this message]
2003-10-04 17:43   ` Andrew Cagney
2003-10-06 19:12     ` Kevin Buettner
2003-10-10 20:22       ` Andrew Cagney
2003-10-10 20:25         ` Jason Thorpe
2003-10-10 20:32           ` Andrew Cagney
2003-10-10 20:34             ` Jason Thorpe
2003-10-10 21:01               ` Andrew Cagney
2003-10-10 21:00         ` Kevin Buettner
2003-10-10 21:27           ` Andrew Cagney

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1030922215845.ZM29725@localhost.localdomain \
    --to=kevinb@redhat.com \
    --cc=ac131313@redhat.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=thorpej@wasabisystems.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox