[v2,18/18] Adjust s_copysignl.c regarding code style.

Message ID 1575895568-26473-19-git-send-email-stli@linux.ibm.com
State New
Headers show
Series
  • Use GCC builtins for some math functions if desired.
Related show

Commit Message

Stefan Liebler Dec. 9, 2019, 12:46 p.m.
This patch just adjusts the generic implementation regarding code style.
No functional change.
---
 sysdeps/ieee754/ldbl-128/s_copysignl.c | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

-- 
2.19.1

Comments

Adhemerval Zanella Dec. 10, 2019, 7:16 p.m. | #1
On 09/12/2019 09:46, Stefan Liebler wrote:
> This patch just adjusts the generic implementation regarding code style.

> No functional change.


LGTM, thanks.

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>


> ---

>  sysdeps/ieee754/ldbl-128/s_copysignl.c | 17 +++++++++--------

>  1 file changed, 9 insertions(+), 8 deletions(-)

> 

> diff --git a/sysdeps/ieee754/ldbl-128/s_copysignl.c b/sysdeps/ieee754/ldbl-128/s_copysignl.c

> index 848a184524..a8b0639d2e 100644

> --- a/sysdeps/ieee754/ldbl-128/s_copysignl.c

> +++ b/sysdeps/ieee754/ldbl-128/s_copysignl.c

> @@ -13,7 +13,7 @@

>   * ====================================================

>   */

>  

> -#if defined(LIBM_SCCS) && !defined(lint)

> +#if defined (LIBM_SCCS) && ! defined (lint)

>  static char rcsid[] = "$NetBSD: $";

>  #endif

>  

> @@ -29,18 +29,19 @@ static char rcsid[] = "$NetBSD: $";

>  #include <libm-alias-ldouble.h>

>  #include <math-use-builtins.h>

>  

> -_Float128 __copysignl(_Float128 x, _Float128 y)

> +_Float128

> +__copysignl (_Float128 x, _Float128 y)

>  {

>  #if USE_COPYSIGNL_BUILTIN

>    return __builtin_copysignl (x, y);

>  #else

>    /* Use generic implementation.  */

> -	uint64_t hx,hy;

> -	GET_LDOUBLE_MSW64(hx,x);

> -	GET_LDOUBLE_MSW64(hy,y);

> -	SET_LDOUBLE_MSW64(x,(hx&0x7fffffffffffffffULL)

> -			    |(hy&0x8000000000000000ULL));

> -        return x;

> +  uint64_t hx, hy;

> +  GET_LDOUBLE_MSW64 (hx, x);

> +  GET_LDOUBLE_MSW64 (hy, y);

> +  SET_LDOUBLE_MSW64 (x, (hx & 0x7fffffffffffffffULL)

> +		     | (hy & 0x8000000000000000ULL));

> +  return x;

>  #endif /* ! USE_COPYSIGNL_BUILTIN  */

>  }

>  libm_alias_ldouble (__copysign, copysign)

> 


Ok.

Patch

diff --git a/sysdeps/ieee754/ldbl-128/s_copysignl.c b/sysdeps/ieee754/ldbl-128/s_copysignl.c
index 848a184524..a8b0639d2e 100644
--- a/sysdeps/ieee754/ldbl-128/s_copysignl.c
+++ b/sysdeps/ieee754/ldbl-128/s_copysignl.c
@@ -13,7 +13,7 @@ 
  * ====================================================
  */
 
-#if defined(LIBM_SCCS) && !defined(lint)
+#if defined (LIBM_SCCS) && ! defined (lint)
 static char rcsid[] = "$NetBSD: $";
 #endif
 
@@ -29,18 +29,19 @@  static char rcsid[] = "$NetBSD: $";
 #include <libm-alias-ldouble.h>
 #include <math-use-builtins.h>
 
-_Float128 __copysignl(_Float128 x, _Float128 y)
+_Float128
+__copysignl (_Float128 x, _Float128 y)
 {
 #if USE_COPYSIGNL_BUILTIN
   return __builtin_copysignl (x, y);
 #else
   /* Use generic implementation.  */
-	uint64_t hx,hy;
-	GET_LDOUBLE_MSW64(hx,x);
-	GET_LDOUBLE_MSW64(hy,y);
-	SET_LDOUBLE_MSW64(x,(hx&0x7fffffffffffffffULL)
-			    |(hy&0x8000000000000000ULL));
-        return x;
+  uint64_t hx, hy;
+  GET_LDOUBLE_MSW64 (hx, x);
+  GET_LDOUBLE_MSW64 (hy, y);
+  SET_LDOUBLE_MSW64 (x, (hx & 0x7fffffffffffffffULL)
+		     | (hy & 0x8000000000000000ULL));
+  return x;
 #endif /* ! USE_COPYSIGNL_BUILTIN  */
 }
 libm_alias_ldouble (__copysign, copysign)