From: Eli Zaretskii <eliz@is.elta.co.il>
To: Jim Blandy <jimb@zwingli.cygnus.com>
Cc: gdb-patches@sources.redhat.com, msnyder@cygnus.com
Subject: Re: RFA: don't try to compare IEEE NaN's
Date: Wed, 06 Jun 2001 08:46:00 -0000 [thread overview]
Message-ID: <Pine.SUN.3.91.1010606184023.18730A-100000@is> (raw)
In-Reply-To: <np66e965u5.fsf@zwingli.cygnus.com>
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?
next prev parent reply other threads:[~2001-06-06 8:46 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 [this message]
2001-06-06 9:15 ` Fernando Nasser
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=Pine.SUN.3.91.1010606184023.18730A-100000@is \
--to=eliz@is.elta.co.il \
--cc=gdb-patches@sources.redhat.com \
--cc=jimb@zwingli.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