This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] Do not provide inlines for GCC build-in functions
- From: "Carlos O'Donell" <carlos at systemhalted dot org>
- To: Andreas Jaeger <aj at suse dot de>
- Cc: libc-alpha at sources dot redhat dot com
- Date: Thu, 17 May 2012 00:12:04 -0400
- Subject: Re: [PATCH] Do not provide inlines for GCC build-in functions
- References: <873971xmm5.fsf@linux-6dv9.site>
On Tue, May 15, 2012 at 3:36 PM, Andreas Jaeger <aj@suse.de> wrote:
>
> As mentioned previously
> (http://sourceware.org/ml/libc-alpha/2012-05/msg00106.html), GCC 3.4
> introduced builtin functions and thus the inlines are not needed at all.
>
> Patch was tested on Linux/x86-64 and I compared libc.so and libm.so and
> found no difference in the assembly.
>
> Looking at math/test-idouble I found a couple of places where GCC used
> now the builtins - instead of a function call like previously. So, GCC
> does a better job than the inlines.
>
> Ok to commit?
>
> Andreas
>
> 2012-05-15 ?Andreas Jaeger ?<aj@suse.de>
>
> ? ? ? ?* sysdeps/x86_64/fpu/bits/mathinline.h (lrintf): Make inline
> ? ? ? ?function only available for GCCs before 3.4 since GCC 3.4
> ? ? ? ?introduced a builtin.
> ? ? ? ?(lrint): Likewise.
> ? ? ? ?(llrintf): Likewise.
> ? ? ? ?(llrint): Likewise.
> ? ? ? ?(fmaxf): Likewise.
> ? ? ? ?(fmax): Likewise.
> ? ? ? ?(fminf): Likewise.
> ? ? ? ?(fmin): Likewise.
> ? ? ? ?(rint): Likewise.
> ? ? ? ?(rintf): Likewise.
> ? ? ? ?(nearbyint): Likewise.
> ? ? ? ?(nearbyintf): Likewise.
> ? ? ? ?(ceil): Likewise.
> ? ? ? ?(ceilf): Likewise.
> ? ? ? ?(floor): Likewise.
> ? ? ? ?(floorf): Likewise.
So what happens if I call lrint but compile with -fno-builtins?
Cheers,
Carlos.