From: Fernando Nasser <fnasser@redhat.com>
To: Eli Zaretskii <eliz@is.elta.co.il>
Cc: Jim Blandy <jimb@cygnus.com>,
gdb-patches@sources.redhat.com, msnyder@cygnus.com
Subject: Re: RFA: don't try to compare IEEE NaN's
Date: Wed, 06 Jun 2001 09:15:00 -0000 [thread overview]
Message-ID: <3B1E5659.6950D735@redhat.com> (raw)
In-Reply-To: <Pine.SUN.3.91.1010606184023.18730A-100000@is>
Eli Zaretskii wrote:
>
> On 6 Jun 2001, Jim Blandy wrote:
>
> > > I think it is better to initialize the integral members of the union
> > > with an explicit bit pattern, just not a pattern which gets
> > > interpreted as a NaN or an Inf. With initialization such as above,
> > > you risk losing due to subtleties of compile-time conversion of a
> > > literal constant to a floating-point value. This is a GDB test suite,
> > > so we are not interested in testing the compiler.
> >
> > I'm not sure what you mean. Once the test has assigned a value to
> > testval.float_testval, we only use that variable. The compile-time
> > conversion happens exactly once, and then we always use the result of
> > that conversion.
>
> Let's begin by asking why do you at all set the members of the union to
> specific values? Why not initialize them to zero, or even leave them at
> some random garbage they pick up from the stack?
>
> My assumption was that whoever wrote the test wanted to see that GDB
> doesn't lose bits due to all kinds of conversions that are going under
> the hood. If that is true, you want to make sure the value you work with
> has the same bit pattern you wanted it to have. If not, you don't really
> know what you are testing here; for example, imagine an (absurdly
> unrealistic) case that the compiler turns your literal constant into an
> all-zero bit pattern, or into a NaN. Then you are back to square one.
>
> The only way to make sure you get the bit patterns you wanted is to
> initialize the integral members of the union with those bit patterns.
> You just want them to be different from a NaN or an Inf, because they
> cause trouble in comparisons.
>
> Am I making any sense?
I believe there will be many failures before that happen.
But, yes, loading the variables with known bit patterns would be immune
to that. On the other hand, what would be the bit pattern? If we use
the IEEE one it may break in systems that do not use IEEE.
--
Fernando Nasser
Red Hat Canada Ltd. E-Mail: fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario M4P 2C9
next prev parent reply other threads:[~2001-06-06 9:15 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-06-05 20:40 Jim Blandy
2001-06-05 23:14 ` Eli Zaretskii
2001-06-06 6:54 ` Fernando Nasser
2001-06-06 7:27 ` Jim Blandy
2001-06-06 8:46 ` Eli Zaretskii
2001-06-06 9:15 ` Fernando Nasser [this message]
2001-06-06 11:38 ` Jim Blandy
2001-06-06 11:52 ` Fernando Nasser
2001-06-06 13:59 ` Michael Snyder
2001-06-06 23:20 ` Eli Zaretskii
2001-06-06 23:19 ` Eli Zaretskii
2001-06-06 10:37 ` Jim Blandy
2001-06-06 10:55 ` Michael Snyder
2001-06-06 11:04 ` Fernando Nasser
2001-06-06 11:15 ` Michael Snyder
2001-06-06 15:08 ` Jim Blandy
2001-06-06 23:19 ` Eli Zaretskii
2001-06-06 23:19 ` Eli Zaretskii
2001-06-07 10:38 ` Michael Snyder
2001-06-07 11:38 ` Eli Zaretskii
2001-06-06 13:17 ` Michael Meissner
2001-06-06 13:45 ` Fernando Nasser
2001-06-06 23:21 ` Eli Zaretskii
2001-06-06 6:45 ` Fernando Nasser
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=3B1E5659.6950D735@redhat.com \
--to=fnasser@redhat.com \
--cc=eliz@is.elta.co.il \
--cc=gdb-patches@sources.redhat.com \
--cc=jimb@cygnus.com \
--cc=msnyder@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