Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Richard Henderson <rth@redhat.com>
To: Andrew Cagney <ac131313@redhat.com>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [rfa/6.0] Better handle unspecified CFI values
Date: Sun, 07 Sep 2003 21:17:00 -0000	[thread overview]
Message-ID: <20030907211748.GA14431@redhat.com> (raw)
In-Reply-To: <3F593115.4030407@redhat.com>

On Fri, Sep 05, 2003 at 08:57:57PM -0400, Andrew Cagney wrote:
> +	  /* Can things get worse?  Yep!  One of the registers GCC
> +	     forgot to provide unwind information for was the stack
> +	     pointer.  Outch!  GCC appears to assumes that the CFA
> +	     address can be used - after all it points to the inner
> +	     most address of the previous frame before the function
> +	     call and that's always the same as the stack pointer on
> +	     return, right?

Yes, gcc does assume the CFA can be used, except when there is
a CFI entry for the stack pointer.  See s390 for this case.

> 	     Wrong.  See GCC's i386 STDCALL option for
> +	     an ABI that has a different entry and return stack

Indeed, this is a problem.  Please file a gcc pr for this.  We
should be generating an entry for the stack pointer in this case.

We probably don't see the bug in gcc's unwinding for exception
handling because we also apply the fixup for DW_CFA_GNU_args_size,
since we're not unwinding to the call site exactly, but rather to
the handler within the function.


r~


      parent reply	other threads:[~2003-09-07 21:17 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-09-06  0:57 Andrew Cagney
2003-09-06 21:34 ` Daniel Jacobowitz
2003-09-09  3:00   ` Andrew Cagney
2003-09-09  3:30     ` Daniel Jacobowitz
2003-09-09 17:24       ` Jim Blandy
2003-09-09 17:35         ` Daniel Jacobowitz
2003-09-10 19:48       ` Richard Henderson
2003-09-10 19:53         ` Daniel Jacobowitz
2003-09-10 21:03           ` Richard Henderson
2003-09-07 20:13 ` Mark Kettenis
2003-09-09  3:00   ` Andrew Cagney
2003-09-07 21:17 ` Richard Henderson [this message]

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=20030907211748.GA14431@redhat.com \
    --to=rth@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