Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Mark Kettenis <mark.kettenis@xs4all.nl>
To: nathan@codesourcery.com
Cc: gdb-patches@sourceware.org, schwab@suse.de
Subject: Re: [m68k]  return values
Date: Mon, 19 Jun 2006 15:12:00 -0000	[thread overview]
Message-ID: <200606191510.k5JFAfG0028604@elgar.sibelius.xs4all.nl> (raw)
In-Reply-To: <4496B738.1050500@codesourcery.com> (message from Nathan Sidwell 	on Mon, 19 Jun 2006 15:39:52 +0100)

> Date: Mon, 19 Jun 2006 15:39:52 +0100
> From: Nathan Sidwell <nathan@codesourcery.com>
> 
> Mark Kettenis wrote:
> 
> > OK, I see your problem now.  The uCLinux ABI is quite different from
> > the normal Linux ABI, and different from the "standard" embedded ABI
> > too.  If I read the GCC code correctly, on uCLinux you end up with:
> 
> I fear I have confused things with the configure.tgt fragment.  The
> main bulk of the patch was to make function return work for
> m68k-elf, which returns pointers in %d0.

Well, I fear you've managed to confuse yourself.  The current m68k
default target uses m68k_extract_return_value(), which reads all
return values from %d0.

> As I understood gdb's internals, we should default to that ABI and
> use OS sniffers to set things up for OS ABIs that are different.

Indeed, although you can override the default ABI by setting gdb_osabi.

> Is the patch ok without the configure.tgt fragment.  I can deal with uclinux 
> separately.

I believe your patch is wrong and unneeded; dealing with uCLinux is
all you need to do.  And I'm afraid it might be better to start doing
that from scratch based on my comments.  Can't really blame you for
being confused though.  The various m68k ABI's are largely
undocumented and the GCC implementation isn't exactly very clear.

Mark


  reply	other threads:[~2006-06-19 15:12 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-09 13:05 Nathan Sidwell
2006-06-18  1:23 ` Mark Kettenis
2006-06-18  5:51 ` Mark Kettenis
2006-06-19 14:39   ` Nathan Sidwell
2006-06-19 15:12     ` Mark Kettenis [this message]
2006-06-19 15:28       ` Nathan Sidwell
2006-06-19 18:32         ` Mark Kettenis
2006-06-22 11:02           ` Nathan Sidwell

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=200606191510.k5JFAfG0028604@elgar.sibelius.xs4all.nl \
    --to=mark.kettenis@xs4all.nl \
    --cc=gdb-patches@sourceware.org \
    --cc=nathan@codesourcery.com \
    --cc=schwab@suse.de \
    /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