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/16928] cacos wrong sign of zero in FE_DOWNWARD mode


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

--- 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  01dbacd22a8d5e0053f8d0cf13a80286b6cfe79b (commit)
      from  913d03c864ea2547e97f8d2d30fc71a008d4d103 (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=01dbacd22a8d5e0053f8d0cf13a80286b6cfe79b

commit 01dbacd22a8d5e0053f8d0cf13a80286b6cfe79b
Author: Joseph Myers <joseph@codesourcery.com>
Date:   Wed May 14 12:37:24 2014 +0000

    Fix cacos (+Inf + finite*i) in round-downward mode (bug 16928).

    According to C99/C11 Annex G, cacos applied to a value with real part
    +Inf and finite imaginary part should produce a result with real part
    +0.  glibc wrongly produces a result with real part -0 in FE_DOWNWARD
    mode.  This patch fixes this by checking for zero results in the
    relevant case of non-finite arguments (where there should never be a
    result with -0 real part), and converts the tests of cacos to
    ALL_RM_TEST.

    Tested x86_64 and x86 and ulps updated accordingly.

        [BZ #16928]
        * math/s_cacos.c (__cacos): Ensure zero real part of result from
        non-finite arguments is +0.
        * math/s_cacosf.c (__cacosf): Likewise.
        * math/s_cacosl.c (__cacosl): Likewise.
        * math/libm-test.inc (cacos_test): Use ALL_RM_TEST.
        * sysdeps/i386/fpu/libm-test-ulps: Update.
        * sysdeps/x86_64/fpu/libm-test-ulps: Likewise.

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

Summary of changes:
 ChangeLog                         |    9 +++++++
 NEWS                              |    2 +-
 math/libm-test.inc                |    4 +--
 math/s_cacos.c                    |    2 +
 math/s_cacosf.c                   |    2 +
 math/s_cacosl.c                   |    2 +
 sysdeps/i386/fpu/libm-test-ulps   |   48 +++++++++++++++++++++++++++++++++++++
 sysdeps/x86_64/fpu/libm-test-ulps |   48 +++++++++++++++++++++++++++++++++++++
 8 files changed, 113 insertions(+), 4 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]