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/19592] [ldbl-128ibm] ceill incorrect in non-default rounding modes


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

--- Comment #2 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  8a9fa0086dfe1b2b9d828712ef41fc7dd4b8f94c (commit)
      from  1833769e19236119d6e41041a53e0f8047156c24 (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=8a9fa0086dfe1b2b9d828712ef41fc7dd4b8f94c

commit 8a9fa0086dfe1b2b9d828712ef41fc7dd4b8f94c
Author: Joseph Myers <joseph@codesourcery.com>
Date:   Thu Feb 18 21:40:39 2016 +0000

    Fix ldbl-128ibm ceill for non-default rounding modes (bug 19592).

    The ldbl-128ibm implementation of ceill is only correct in
    round-to-nearest mode (in other modes, there are incorrect results and
    overflow exceptions in some cases).  It is also unnecessarily
    complicated, rounding both high and low parts to the nearest integer
    and then adjusting for the semantics of ceil, when it seems more
    natural to take the ceiling of the high part (__ceil optimized inline
    versions can be used), and that of the low part if the high part is an
    integer, as was done for floorl.  This patch makes it use that simpler
    approach.

    Tested for powerpc.

        [BZ #19592]
        * sysdeps/ieee754/ldbl-128ibm/s_ceill.c (__ceill): Use __ceil on
        high and low parts then use ldbl_canonicalize_int if needed.

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

Summary of changes:
 ChangeLog                             |    4 ++
 sysdeps/ieee754/ldbl-128ibm/s_ceill.c |   52 ++++++++++----------------------
 2 files changed, 20 insertions(+), 36 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]