This is the mail archive of the glibc-bugs@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug math/20357] Incorrect cos result for 1.5174239687223976


https://sourceware.org/bugzilla/show_bug.cgi?id=20357

--- Comment #1 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".

The branch, master has been updated
       via  cbf88869edced4b23d792d95a8626e35b831df35 (commit)
      from  2413e73c32fc36470885ae548631e081d66f4201 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://sourceware.org/git/gitweb.cgi?p=glibc.git;h=cbf88869edced4b23d792d95a8626e35b831df35

commit cbf88869edced4b23d792d95a8626e35b831df35
Author: Siddhesh Poyarekar <siddhesh@sourceware.org>
Date:   Mon Jul 18 22:33:09 2016 +0530

    Fix cos computation for multiple precision fallback (bz #20357)

    During the sincos consolidation I made two mistakes, one was a logical
    error due to which cos(0x1.8475e5afd4481p+0) returned
    sin(0x1.8475e5afd4481p+0) instead.

    The second issue was an error in negating inputs for the correct
    quadrants for sine.  I could not find a suitable test case for this
    despite running a program to search for such an input for a couple of
    hours.

    Following patch fixes both issues.  Tested on x86_64.  Thanks to Matt
    Clay for identifying the issue.

        [BZ #20357]
        * sysdeps/ieee754/dbl-64/s_sin.c (sloww): Fix up condition
        to call __mpsin/__mpcos and to negate values.
        * math/auto-libm-test-in: Add test.
        * math/auto-libm-test-out: Regenerate.

-----------------------------------------------------------------------

Summary of changes:
 ChangeLog                      |    9 ++
 math/auto-libm-test-in         |    3 +
 math/auto-libm-test-out        |  207 ++++++++++++++++++++++++++++++++++++++++
 sysdeps/ieee754/dbl-64/s_sin.c |    4 +-
 4 files changed, 221 insertions(+), 2 deletions(-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]