diff options
| author | Siddhesh Poyarekar <siddhesh@redhat.com> | 2014-02-27 21:29:16 +0530 |
|---|---|---|
| committer | Siddhesh Poyarekar <siddhesh@redhat.com> | 2014-02-27 21:29:16 +0530 |
| commit | ffe768a90912f9bce43b70a82576b3dc99e3121c (patch) | |
| tree | e5b2d4c110c959fdd21974d2afde892823eec049 | |
| parent | ac0b347743833452b8ff56cb94eab5da95136b70 (diff) | |
| download | glibc-ffe768a90912f9bce43b70a82576b3dc99e3121c.tar.gz | |
Fix sign of input to bsloww1 (BZ #16623)
In 84ba214c, I removed some redundant sign computations and in the
process, I incorrectly got rid of a temporary variable, thus passing
the absolute value of the input to bsloww1. This caused #16623.
This fix undoes the incorrect change.
| -rw-r--r-- | ChangeLog | 7 | ||||
| -rw-r--r-- | NEWS | 2 | ||||
| -rw-r--r-- | sysdeps/ieee754/dbl-64/s_sin.c | 16 |
3 files changed, 18 insertions, 7 deletions
@@ -1,3 +1,10 @@ +2014-02-27 Siddhesh Poyarekar <siddhesh@redhat.com> + + [BZ #16623] + * sysdeps/ieee754/dbl-64/s_sin.c (__sin): Preserve sign of A + and DA. + (__cos): Likewise. + 2014-02-12 Dylan Alex Simon <dylan@dylex.net> [BZ #16545] @@ -9,7 +9,7 @@ Version 2.19.1 * The following bugs are resolved with this release: - 16545. + 16545, 16623. Version 2.19 diff --git a/sysdeps/ieee754/dbl-64/s_sin.c b/sysdeps/ieee754/dbl-64/s_sin.c index 6105e9fbdf..50109b8dd4 100644 --- a/sysdeps/ieee754/dbl-64/s_sin.c +++ b/sysdeps/ieee754/dbl-64/s_sin.c @@ -447,19 +447,21 @@ __sin (double x) } else { + double t; if (a > 0) { m = 1; + t = a; db = da; } else { m = 0; - a = -a; + t = -a; db = -da; } - u.x = big + a; - y = a - (u.x - big); + u.x = big + t; + y = t - (u.x - big); res = do_sin (u, y, db, &cor); cor = (cor > 0) ? 1.035 * cor + eps : 1.035 * cor - eps; retval = ((res == res + cor) ? ((m) ? res : -res) @@ -671,19 +673,21 @@ __cos (double x) } else { + double t; if (a > 0) { m = 1; + t = a; db = da; } else { m = 0; - a = -a; + t = -a; db = -da; } - u.x = big + a; - y = a - (u.x - big); + u.x = big + t; + y = t - (u.x - big); res = do_sin (u, y, db, &cor); cor = (cor > 0) ? 1.035 * cor + eps : 1.035 * cor - eps; retval = ((res == res + cor) ? ((m) ? res : -res) |