diff options
author | Joseph Myers <joseph@codesourcery.com> | 2013-04-02 22:54:00 +0000 |
---|---|---|
committer | Joseph Myers <joseph@codesourcery.com> | 2013-04-02 22:54:00 +0000 |
commit | 52ce486045074d0af0298082f94e385e6b2fe443 (patch) | |
tree | 472a67e3c1b2881643aaa315cbfc32fee80e38a0 /math/s_cacoshf.c | |
parent | e7906a4789f9f6ac4cd4a99522832753887a8cc7 (diff) | |
download | glibc-52ce486045074d0af0298082f94e385e6b2fe443.tar.gz |
Fix cacosh inaccuracy and spurious exceptions (bug 15327).
Diffstat (limited to 'math/s_cacoshf.c')
-rw-r--r-- | math/s_cacoshf.c | 33 |
1 files changed, 13 insertions, 20 deletions
diff --git a/math/s_cacoshf.c b/math/s_cacoshf.c index 74cb2760d6..1e692f139e 100644 --- a/math/s_cacoshf.c +++ b/math/s_cacoshf.c @@ -64,32 +64,25 @@ __cacoshf (__complex__ float x) __real__ res = 0.0; __imag__ res = __copysignf (M_PI_2, __imag__ x); } - /* The factor 16 is just a guess. */ - else if (16.0 * fabsf (__imag__ x) < fabsf (__real__ x)) - { - /* Kahan's formula which avoid cancellation through subtraction in - some cases. */ - res = 2.0 * __clogf (__csqrtf ((x + 1.0) / 2.0) - + __csqrtf ((x - 1.0) / 2.0)); - if (signbit (__real__ res)) - __real__ res = 0.0f; - } else { __complex__ float y; - __real__ y = (__real__ x - __imag__ x) * (__real__ x + __imag__ x) - 1.0; - __imag__ y = 2.0 * __real__ x * __imag__ x; - - y = __csqrtf (y); + __real__ y = -__imag__ x; + __imag__ y = __real__ x; - if (signbit (__real__ x)) - y = -y; + y = __kernel_casinhf (y, 1); - __real__ y += __real__ x; - __imag__ y += __imag__ x; - - res = __clogf (y); + if (signbit (__imag__ x)) + { + __real__ res = __real__ y; + __imag__ res = -__imag__ y; + } + else + { + __real__ res = -__real__ y; + __imag__ res = __imag__ y; + } } return res; |