diff options
| author | Joseph Myers <joseph@codesourcery.com> | 2015-10-09 00:32:14 +0000 |
|---|---|---|
| committer | Joseph Myers <joseph@codesourcery.com> | 2015-10-09 00:32:14 +0000 |
| commit | 0e069029a8bb132876d315242054a312ae106852 (patch) | |
| tree | 3afba08492052af6c50456621e0aba3323bccc29 /sysdeps/ieee754/dbl-64/s_lrint.c | |
| parent | d0d286d32dda654f8983e8fe77bca0a2cda2051b (diff) | |
| download | glibc-0e069029a8bb132876d315242054a312ae106852.tar.gz glibc-0e069029a8bb132876d315242054a312ae106852.tar.xz glibc-0e069029a8bb132876d315242054a312ae106852.zip | |
Fix dbl-64 lrint for 64-bit long (bug 19095).
The dbl-64 implementation of lrint produces incorrect results for some arguments with 64-bit long because a 32-bit (unsigned) low part of the mantissa is shifted left, losing high bits in the process. This patch fixes this by casting to long int before shifting, as in lround (as this case only applies for 64-bit long, there are no issues with sign-extension). Tested for mips64 (n64). [BZ #19095] * sysdeps/ieee754/dbl-64/s_lrint.c (__lrint): Cast low part of mantissa to long int before shifting left.
Diffstat (limited to 'sysdeps/ieee754/dbl-64/s_lrint.c')
| -rw-r--r-- | sysdeps/ieee754/dbl-64/s_lrint.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sysdeps/ieee754/dbl-64/s_lrint.c b/sysdeps/ieee754/dbl-64/s_lrint.c index 39f95adc21..d004594bc2 100644 --- a/sysdeps/ieee754/dbl-64/s_lrint.c +++ b/sysdeps/ieee754/dbl-64/s_lrint.c @@ -61,7 +61,7 @@ __lrint (double x) else if (j0 < (int32_t) (8 * sizeof (long int)) - 1) { if (j0 >= 52) - result = ((long int) i0 << (j0 - 20)) | (i1 << (j0 - 52)); + result = ((long int) i0 << (j0 - 20)) | ((long int) i1 << (j0 - 52)); else { #if defined FE_INVALID || defined FE_INEXACT |