This is the mail archive of the
glibc-bugs@sourceware.org
mailing list for the glibc project.
[Bug math/19593] [ldbl-128ibm] truncl incorrect in non-default rounding modes
- From: "cvs-commit at gcc dot gnu.org" <sourceware-bugzilla at sourceware dot org>
- To: glibc-bugs at sourceware dot org
- Date: Thu, 18 Feb 2016 21:53:09 +0000
- Subject: [Bug math/19593] [ldbl-128ibm] truncl incorrect in non-default rounding modes
- Auto-submitted: auto-generated
- References: <bug-19593-131 at http dot sourceware dot org/bugzilla/>
https://sourceware.org/bugzilla/show_bug.cgi?id=19593
--- 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 e2310a27bede834c7b63a8bd1d659c376b6388df (commit)
from 8a9fa0086dfe1b2b9d828712ef41fc7dd4b8f94c (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=e2310a27bede834c7b63a8bd1d659c376b6388df
commit e2310a27bede834c7b63a8bd1d659c376b6388df
Author: Joseph Myers <joseph@codesourcery.com>
Date: Thu Feb 18 21:52:07 2016 +0000
Fix ldbl-128ibm truncl for non-default rounding modes (bug 19593).
The ldbl-128ibm implementation of truncl 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 trunc, when it seems more
natural to take the truncation of the high part (__trunc optimized
inline versions can be used), and the floor or ceiling of the low part
(depending on the sign of the high part) if the high part is an
integer, as was done for floorl and ceill. This patch makes it use
that simpler approach.
Tested for powerpc.
[BZ #19593]
* sysdeps/ieee754/ldbl-128ibm/s_truncl.c (__truncl): Use __trunc
on high part and __floor or __ceil on low part then use
ldbl_canonicalize_int if needed.
-----------------------------------------------------------------------
Summary of changes:
ChangeLog | 5 +++
sysdeps/ieee754/ldbl-128ibm/s_truncl.c | 50 +++++++-------------------------
2 files changed, 16 insertions(+), 39 deletions(-)
--
You are receiving this mail because:
You are on the CC list for the bug.