diff mbox

[COMMITTED] Use __copysign rather than copysign

Message ID 000201d07cf6$423011f0$c69035d0$@com
State New
Headers show

Commit Message

Wilco April 22, 2015, 12:17 p.m. UTC
I noticed a few uses of copysign in generic GLIBC sources - these should be __copysign
to avoid namespace issues and enable inlining.

Directly checked in as trivial change.

2015-04-22  Wilco Dijkstra  <wdijkstr@arm.com>

        * math/k_casinh.c (__kernel_casinh): Use __copysign.
        * sysdeps/ieee754/dbl-64/wordsize-64/s_nearbyint.c
        (__nearbyint): Likewise.
        * sysdeps/ieee754/ldbl-opt/nldbl-copysign.c
        (copysignl): Likewise.

---
 math/k_casinh.c                                  | 2 +-
 sysdeps/ieee754/dbl-64/wordsize-64/s_nearbyint.c | 2 +-
 sysdeps/ieee754/ldbl-opt/nldbl-copysign.c        | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)
diff mbox

Patch

diff --git a/math/k_casinh.c b/math/k_casinh.c
index de04e84..cd089ab 100644
--- a/math/k_casinh.c
+++ b/math/k_casinh.c
@@ -199,7 +199,7 @@  __kernel_casinh (__complex__ double x, int adj)
       if (adj)
 	{
 	  double t = __real__ y;
-	  __real__ y = copysign (__imag__ y, __imag__ x);
+	  __real__ y = __copysign (__imag__ y, __imag__ x);
 	  __imag__ y = t;
 	}
 
diff --git a/sysdeps/ieee754/dbl-64/wordsize-64/s_nearbyint.c
b/sysdeps/ieee754/dbl-64/wordsize-64/s_nearbyint.c
index a58a620..a81e5ed 100644
--- a/sysdeps/ieee754/dbl-64/wordsize-64/s_nearbyint.c
+++ b/sysdeps/ieee754/dbl-64/wordsize-64/s_nearbyint.c
@@ -46,7 +46,7 @@  __nearbyint(double x)
 		double t =  w-TWO52[sx];
 		math_opt_barrier(t);
 		libc_fesetenv (&env);
-		return copysign(t, x);
+		return __copysign (t, x);
 	    }
 	} else {
 	    if(j0==0x400) return x+x;	/* inf or NaN */
diff --git a/sysdeps/ieee754/ldbl-opt/nldbl-copysign.c b/sysdeps/ieee754/ldbl-opt/nldbl-copysign.c
index ef23bad..045f00d 100644
--- a/sysdeps/ieee754/ldbl-opt/nldbl-copysign.c
+++ b/sysdeps/ieee754/ldbl-opt/nldbl-copysign.c
@@ -4,5 +4,5 @@  double
 attribute_hidden
 copysignl (double x, double y)
 {
-  return copysign (x, y);
+  return __copysign (x, y);
 }