This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Use fabs(f/l) rather than __fabs

From: Joseph Myers <joseph at codesourcery dot com>
To: Wilco Dijkstra <Wilco dot Dijkstra at arm dot com>
Cc: "libc-alpha at sourceware dot org" <libc-alpha at sourceware dot org>, nd <nd at arm dot com>
Date: Fri, 29 Sep 2017 19:56:44 +0000
Subject: Re: [PATCH] Use fabs(f/l) rather than __fabs
Authentication-results: sourceware.org; auth=none
References: <DB6PR0801MB2053B16BF79A3E6ECD6D7359837E0@DB6PR0801MB2053.eurprd08.prod.outlook.com>,<alpine.DEB.2.20.1709291623180.25795@digraph.polyomino.org.uk> <DB6PR0801MB2053ADFC56A678C3C97DAFBB837E0@DB6PR0801MB2053.eurprd08.prod.outlook.com>

On Fri, 29 Sep 2017, Wilco Dijkstra wrote:

> Right so there are no namespace issues. But what about localplt issues
> due to the disabling of fabsl inlining? I can't find any renaming, so why are
> there no local plt failures on ppc soft-float?

Because both fabsl and copysignl are listed in 
sysdeps/unix/sysv/linux/powerpc/powerpc32/nofpu/localplt.data.  If the GCC 
bug is fixed and the -fno-builtin-fabsl use made conditional, then fabsl 
would also need to be marked optional in localplt.data as copysignl is.

-- 
Joseph S. Myers
joseph@codesourcery.com

References:
- [PATCH] Use fabs(f/l) rather than __fabs
  - From: Wilco Dijkstra
- Re: [PATCH] Use fabs(f/l) rather than __fabs
  - From: Joseph Myers
- Re: [PATCH] Use fabs(f/l) rather than __fabs
  - From: Wilco Dijkstra

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]