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
Subject: Re: [rfa:ppc64] Fix 64-bit PPC ELF function calls
Date: Fri, 03 Oct 2003 21:22:00 -0000	[thread overview]
Message-ID: <1031003212231.ZM26624@localhost.localdomain> (raw)
In-Reply-To: Andrew Cagney <ac131313@redhat.com> "Re: [rfa:ppc64] Fix 64-bit PPC ELF function calls" (Sep 22,  1:59pm)

Sorry for not replying sooner.  I thought I had replied to this already...

On Sep 22,  1:59pm, Andrew Cagney wrote:

> > PS: The apparent bugs are:
> > 
> > - small odd structs get passed in memory instead of a register
> >   (ref structs.exp:Fun3).
> > - small even structs get passed right, instead of left, aligned in
> >   the register (ref structs.exp:Fun[12]).

These are all GCC bugs, right?  And, if they get fixed in GCC, then GDB
will be broken?

> > PS: Backtraces are a bit sick.

In what way?

> > PPS:  Oh, note the "hack" to find the TOC from the function's
> >       entry point address.  Without it malloc() fails.

Calls to other library functions would fail too.

Is the TOC symbol name vs entry point name (i.e, FN vs..FN) convention
mandated by the ABI, or is this something that's Linux specific?

I notice some 80+ character lines in ppc64_sysv_abi_push_dummy_call().
Could you adjust these so that they're 80 characters or less?

Also, a minor nit: in the comment...

  /* Find a value for the TOC register.  Every symbol should have both
     ".FN" and "FN" in the minimal symbol table.  "FN" points at the
     F's descriptor, while ".FN" points at the entry point (which
     matches FUNC_ADDR).  Need to reverse from FUNC_ADDR back to the
     FN's descriptor address.  */

...at the beginning of the third line down, shouldn't that be:

     FN's descriptor, [...]

If not, what does `F' refer to?

Kevin


  parent reply	other threads:[~2003-10-03 21:22 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-09-21 23:38 Andrew Cagney
2003-09-22 17:59 ` Andrew Cagney
2003-10-03 20:48   ` Andrew Cagney
2003-10-03 21:22   ` Kevin Buettner [this message]
2003-10-06 22:19     ` Andrew Cagney
2003-10-09  5:02       ` Kevin Buettner
2003-10-10  1:27         ` Andrew Cagney
2003-10-10  5:20           ` Kevin Buettner
2003-10-10 18:30             ` 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=1031003212231.ZM26624@localhost.localdomain \
    --to=kevinb@redhat.com \
    --cc=ac131313@redhat.com \
    --cc=gdb-patches@sources.redhat.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