PATCH: Use __builtin_clzll in 64-bit __logb

Andreas Jaeger aj@suse.com
Mon May 14 19:13:00 GMT 2012


On 05/11/2012 02:22 PM, H.J. Lu wrote:
> Hi,
>
> X32 uses 64-bit math function.  But its long is 32bit.  64-bit __logb
> should use __builtin_clzll instead of __builtin_clzl.  Testd on Linux/x32
> and Linux/x86-64.  OK to install?

HJ, you filed bz # 14103 for this. Please add the number to ChangeLog, 
to NEWS and mark it as resolved (the same applies for 14104),

Andreas

> Thanks.
>
> H.J.
> ---
> 	* sysdeps/ieee754/dbl-64/wordsize-64/s_logb.c (__logb): Replace
> 	__builtin_clzl with __builtin_clzll.
>
> diff --git a/sysdeps/ieee754/dbl-64/wordsize-64/s_logb.c b/sysdeps/ieee754/dbl-64/wordsize-64/s_logb.c
> index 474eeef..185dd05 100644
> --- a/sysdeps/ieee754/dbl-64/wordsize-64/s_logb.c
> +++ b/sysdeps/ieee754/dbl-64/wordsize-64/s_logb.c
> @@ -36,7 +36,7 @@ __logb (double x)
>       return x * x;
>     if (__builtin_expect (ex == 0, 0))
>       {
> -      int m = (ix == 0) ? 0 : __builtin_clzl (ix);
> +      int m = (ix == 0) ? 0 : __builtin_clzll (ix);
>         return -1022.0 + (double)(11 -m);
>       }
>     return (double) (ex - 1023);


-- 
  Andreas Jaeger aj@{suse.com,opensuse.org} Twitter/Identica: jaegerandi
   SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
    GF: Jeff Hawn,Jennifer Guild,Felix Imendörffer,HRB16746 (AG Nürnberg)
     GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126



More information about the Libc-alpha mailing list