Use Kahan's formula in cacosh
authorUlrich Drepper <drepper@gmail.com>
Thu, 22 Dec 2011 03:08:12 +0000 (22:08 -0500)
committerUlrich Drepper <drepper@gmail.com>
Thu, 22 Dec 2011 03:08:12 +0000 (22:08 -0500)
ChangeLog
math/s_cacosh.c
math/s_cacoshf.c
math/s_cacoshl.c

index 1ccc3dd..4d2b1fc 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2011-12-21  Ulrich Drepper  <drepper@gmail.com>
+
+       * math/s_cacosh.c: Use Kahan's formula if the subtraction could lead
+       to large cancellation.
+       * math/s_cacoshf.c: Likewise.
+       * math/s_cacoshl.c: Likewise.
+
 2011-11-18  Richard B. Kreckel  <kreckel@ginac.de>
 
        [BZ #13305]
index df5ce69..cb01ca8 100644 (file)
@@ -65,6 +65,11 @@ __cacosh (__complex__ double x)
       __real__ res = 0.0;
       __imag__ res = __copysign (M_PI_2, __imag__ x);
     }
+  /* The factor 16 is just a guess.  */
+  else if (16.0 * fabs (__imag__ x) < fabs (__real__ x))
+    /* Kahan's formula which avoid cancellation through subtraction in
+       some cases.  */
+    res = 2.0 * __clog (__csqrt ((x + 1.0) / 2.0) + __csqrt ((x - 1.0) / 2.0));
   else
     {
       __complex__ double y;
index aa4696f..fe04692 100644 (file)
@@ -65,6 +65,12 @@ __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));
   else
     {
       __complex__ float y;
index b90b196..84c2715 100644 (file)
@@ -65,6 +65,12 @@ __cacoshl (__complex__ long double x)
       __real__ res = 0.0;
       __imag__ res = __copysignl (M_PI_2l, __imag__ x);
     }
+  /* The factor 16 is just a guess.  */
+  else if (16.0L * fabsl (__imag__ x) < fabsl (__real__ x))
+    /* Kahan's formula which avoid cancellation through subtraction in
+       some cases.  */
+    res = 2.0L * __clogl (__csqrtl ((x + 1.0L) / 2.0L)
+                         + __csqrtl ((x - 1.0L) / 2.0L));
   else
     {
       __complex__ long double y;