public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: "John David Anglin" <dave@hiauly1.hia.nrc.ca>
To: law@cygnus.com
Cc: meissner@cygnus.com, gcc-patches@gcc.gnu.org
Subject: Re: PATCH: HUGE_VAL should be Infinity
Date: Tue, 17 Oct 2000 11:51:00 -0000	[thread overview]
Message-ID: <200010171851.OAA15821@hiauly1.hia.nrc.ca> (raw)
In-Reply-To: <18723.971798047@upchuck>

>   In message < 200010170458.AAA23638@hiauly1.hia.nrc.ca >you write:
>   > Having thought a little more about this, I believe the change is wrong as i
>   > will make the define in math.h inconsistent with the system's existing math
>   > library.  Thus, the patch should not be applied.
> Excellent point.
> 
> I guess the "fix" is to upgrade to hpux11 where this situation is handled
> more gracefully.  FWIW, I do most of my hpux work on hpux11 now.

After thinking about this overnight, I beieve the hugeval.c test should be
changed because the second half is not a compiler test.  It checks whether
the definition of HUGE_VAL is +Infinity.  The linux math.h header states
that HUGE_VAL is +Infinity on all IEEE754 machines.  However, this appears
not to be the case.  In any case, systems predating the standard or without
ieee floating point will fail the hugeval test.  Since I don't think
compiler tests should fail when there is nothing wrong with the compiler,
I will try to come up with a test fix.

Dave
-- 
J. David Anglin                                  dave.anglin@nrc.ca
National Research Council of Canada              (613) 990-0752 (FAX: 952-6605)

  reply	other threads:[~2000-10-17 11:51 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-10-16 10:46 John David Anglin
2000-10-16 18:35 ` Jeffrey A Law
2000-10-16 19:02   ` John David Anglin
2000-10-17  8:35     ` Jeffrey A Law
2000-10-16 20:21   ` Michael Meissner
2000-10-16 21:58     ` John David Anglin
2000-10-17  8:53       ` Jeffrey A Law
2000-10-17 11:51         ` John David Anglin [this message]
2000-10-17 18:51           ` Jeffrey A Law
2000-10-17 21:19             ` John David Anglin
2000-11-29 23:31               ` Jeffrey A Law
2000-11-30 10:23                 ` John David Anglin
2000-11-30 10:55                   ` Michael Meissner
2000-11-30 11:36                     ` John David Anglin
2000-10-18 14:03             ` PATCH: HUGE_VAL should be Infinity -- Yet another solution John David Anglin
2000-10-17 16:06         ` PATCH: HUGE_VAL should be Infinity: hugeval.c PATCH John David Anglin
     [not found] <no.id>
2000-11-30 12:16 ` PATCH: HUGE_VAL should be Infinity John David Anglin
2000-12-03 21:14   ` Jeffrey A Law
2000-12-04 10:52     ` Michael Meissner
2000-12-04 11:25       ` John David Anglin
2000-12-04 11:35         ` Jeffrey A Law
2000-12-06 18:05         ` Jeffrey A Law

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=200010171851.OAA15821@hiauly1.hia.nrc.ca \
    --to=dave@hiauly1.hia.nrc.ca \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=law@cygnus.com \
    --cc=meissner@cygnus.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).