public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Jim Wilson <wilson@specifixinc.com>
To: Daniel Jacobowitz <drow@mvista.com>
Cc: gcc-patches@gcc.gnu.org
Subject: Re: avoid unnecessary register saves for setjmp
Date: Sat, 22 Nov 2003 09:06:00 -0000	[thread overview]
Message-ID: <1069490758.1024.191.camel@leaf.tuliptree.org> (raw)
In-Reply-To: <20031122030808.GA23061@nevyn.them.org>

On Fri, 2003-11-21 at 19:08, Daniel Jacobowitz wrote:
> Here's the difficulty: changing the size of jmp_buf.

That is a good point.  Still, we shouldn't penalize targets that have
defined setjmp correctly.  So I suggest using NON_SAVING_SETJMP for
this.  This will do what you want if you define it for your target. 
I'll go ahead and fix NON_SAVING_SETJMP to make it usable.

> Sure - but that assumes that all unused call-saved registers are
> restored by the setjmp.  So then it is required to save registers to
> get reasonable behavior.

You are confusing two different issues here.  setjmp is not required to
restore call-used registers.  Thus any value stored in a register in
this function may not be restored when longjmp is called.  This is why
the C language standard says that local automatic non-volatile variables
may not be restored by longjmp.  However, setjmp is required to restore
the original context, and that means call-saved registers must be
restored.  A setjmp call is not allowed to clobber locals in the parent
function, which is what would happen if call-saved registers were not
restored.
-- 
Jim Wilson, GNU Tools Support, http://www.SpecifixInc.com

  reply	other threads:[~2003-11-22  8:45 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-11-21  5:54 Jim Wilson
2003-11-21 18:39 ` Daniel Jacobowitz
2003-11-22  0:22   ` Jim Wilson
2003-11-22  3:57     ` Daniel Jacobowitz
2003-11-22  9:06       ` Jim Wilson [this message]
2003-11-22 14:37         ` Richard Earnshaw
2003-11-22 16:33           ` Daniel Jacobowitz
2003-11-27  9:11             ` Zack Weinberg
2003-11-27 17:11               ` Andrew Pinski
2003-11-27 18:23                 ` Zack Weinberg
2003-11-28  5:15                   ` Geert Bosch
2003-11-28 11:14                   ` Richard Earnshaw
2003-11-28 14:02             ` Richard Earnshaw
2003-11-21 20:14 ` Geoff Keating
2003-11-22  0:20   ` Jim Wilson
2003-11-30  9:23   ` Jim Wilson
2003-11-22  4:13 Richard Kenner
2003-12-01  3:29 ` Jim Wilson
2003-11-27  9:19 Chris Lattner
2003-11-27  9:21 ` Andrew Haley
2003-11-27  9:22   ` Chris Lattner
2003-11-27  9:39     ` Andrew Haley
2003-11-27  9:43       ` Chris Lattner
2003-11-27 10:14       ` Zack Weinberg
2003-11-27 10:15         ` Chris Lattner
2003-11-27 11:01           ` Zack Weinberg
2003-11-27 20:28             ` Chris Lattner
2003-11-27 20:51               ` Gabriel Dos Reis
2003-11-27 15:41           ` Jan Vroonhof
2003-11-27 16:23           ` Jan Vroonhof
2003-11-27 10:31         ` Andrew Haley
2003-11-27 10:53           ` Zack Weinberg
     [not found] <5cad8ef043da68f2a3332f00bd6a186a3fc6195b@mail.esmertec.com>
2003-11-27 20:44 ` Chris Lattner

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=1069490758.1024.191.camel@leaf.tuliptree.org \
    --to=wilson@specifixinc.com \
    --cc=drow@mvista.com \
    --cc=gcc-patches@gcc.gnu.org \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).