public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Mark Mitchell <mark@codesourcery.com>
To: Geoff Keating <geoffk@geoffk.org>
Cc: "steby@enea.se" <steby@enea.se>,
	"gcc@gcc.gnu.org" <gcc@gcc.gnu.org>,
	zack@codesourcery.com
Subject: Re: GCC floating point usage
Date: Tue, 15 Oct 2002 18:09:00 -0000	[thread overview]
Message-ID: <184220000.1034725921@warlock.codesourcery.com> (raw)
In-Reply-To: <200210152337.g9FNbkG07955@desire.geoffk.org>

> OK, that's a good description.  It's clear, easily understood, and has
> a bunch of consequences I bet you haven't thought of yet :-).

Could well be. :-)

> For instance, consider a program that uses setjmp in one file, which
> contains no use of FP but uses routines from another file that call
> longjmp and do use FP.

I've thought of this one, though.

That's considered user error in VxWorks; no different from reading
from uninitialized memory.

> Note that this doesn't apply to "most code"; assuming vxworks defaults
> to -msoft-float, then -mhard-float is only necessary when it's
> actually necessary to use the FPU.

I said "think about" -- not actually pass the switch.  You still have
to think about in your Makefile, and in a mostly-FP task (say, a
signal processing task), you might have almost-all FP code.

>> So, let's first answer the question: would you accept a patch which had
>> the behavior above, assuming it were otherwise OK?
>
> It's not a black-and-white issue.  A small, simple, obviously correct
> patch that implements the feature would be much more acceptable than a
> large complex buggy patch.

Well, sure -- that was supposed to be the "otherwise OK" bit. :-)

Are you saying that a good patch to implement this feature would be OK,
for appropriate definition of "good"?

If so, then we can go back to the technical discussion about how best
to implement the imaginary flag.

-- 
Mark Mitchell                mark@codesourcery.com
CodeSourcery, LLC            http://www.codesourcery.com

  reply	other threads:[~2002-10-15 23:54 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-10-14  9:59 Stefan Bylund
2002-10-14 10:16 ` David Edelsohn
2002-10-14 10:25   ` Zack Weinberg
2002-10-14 10:55     ` Dale Johannesen
2002-10-14 22:22     ` Mark Mitchell
2002-10-15 15:35       ` Geoff Keating
2002-10-15 16:10         ` Mark Mitchell
2002-10-15 17:20           ` Geoff Keating
2002-10-15 18:09             ` Mark Mitchell [this message]
2002-10-16  7:40               ` Joel Sherrill
2002-10-15 19:04             ` Mike Stump
2002-10-16 12:06               ` Mark Mitchell
2002-10-16 13:35                 ` Geoff Keating
2002-10-16 14:29                   ` Mark Mitchell
2002-10-16 14:56                     ` Michael Matz
2002-10-16 15:03                       ` Mark Mitchell
2002-10-16 15:27                         ` David Edelsohn
2002-10-16 15:36                           ` Mark Mitchell
2002-10-16 16:35                             ` Zack Weinberg
2002-10-16 16:36                               ` Mark Mitchell
2002-10-16 16:46                             ` David Edelsohn
2002-10-17  8:37                             ` Paul Koning
2002-10-16 17:57                           ` Mike Stump
2002-10-17  4:12                   ` Mike Stump
2002-10-16 13:43               ` Richard Henderson
2002-10-16 14:35                 ` Mark Mitchell
2002-10-16 14:56                   ` Joel Sherrill
2002-10-16 16:38                   ` Richard Henderson
2002-10-16 16:53                     ` Zack Weinberg
2002-10-16 17:52                       ` Michael Matz
2002-10-16 22:50                       ` Richard Henderson
2002-10-21 12:21                         ` Jeff Law
2002-10-16 17:29                 ` Mike Stump
2002-10-17  2:19                   ` Richard Henderson
2002-10-15 17:19         ` Mike Stump
2002-10-15 18:41         ` Zack Weinberg
2002-10-16  1:48           ` Fergus Henderson
2002-10-14 10:37   ` Stefan Bylund
2002-10-14 11:28     ` Mike Stump
2002-10-14 12:39       ` Joel Sherrill

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=184220000.1034725921@warlock.codesourcery.com \
    --to=mark@codesourcery.com \
    --cc=gcc@gcc.gnu.org \
    --cc=geoffk@geoffk.org \
    --cc=steby@enea.se \
    --cc=zack@codesourcery.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).