public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "ian at airs dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug go/63269] libgo/math test failures in TestLog2
Date: Wed, 05 Nov 2014 03:54:00 -0000	[thread overview]
Message-ID: <bug-63269-4-CEZECpirRZ@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-63269-4@http.gcc.gnu.org/bugzilla/>

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63269

Ian Lance Taylor <ian at airs dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |WAITING
   Last reconfirmed|                            |2014-11-05
     Ever confirmed|0                           |1

--- Comment #3 from Ian Lance Taylor <ian at airs dot com> ---
First, let me say that this code is in the Go master library and must be fixed
there.  It might be more effective to discuss it on the Go issue tracker at
http://golang.org/issue.

I don't agree with your argument for item 1.  You say that the wrong result
happens when using a fused multiply-add instruction.  The math library is
compiled with -ffp-contract=off (see MATH_FLAG in configure.ac and
Makefile.am), so the compiler should not be generating a fused multiply-add
instruction.

I'm not entirely persuaded by your argument for item 2.  Zero is a special
value.  When we expect a zero, we should get a zero, not something close to
zero.  I don't think this change is correct in general.  It may be correct for
some specific cases, but then we need to investigate those.

Item 3 is the same sort of thing: when we expect zero, we should, in general,
get exactly zero.


  parent reply	other threads:[~2014-11-05  3:54 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-15 14:10 [Bug go/63269] New: " vogt at linux dot vnet.ibm.com
2014-09-15 14:12 ` [Bug go/63269] " vogt at linux dot vnet.ibm.com
2014-09-17  8:35 ` vogt at linux dot vnet.ibm.com
2014-11-05  3:54 ` ian at airs dot com [this message]
2014-11-05 14:34 ` vogt at linux dot vnet.ibm.com
2014-11-05 16:47 ` vogt at linux dot vnet.ibm.com
2014-11-06 10:18 ` vogt at linux dot vnet.ibm.com

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=bug-63269-4-CEZECpirRZ@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@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).