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 v2 1/3] Remove unneeded declarations from math_private.h


On Tue, 28 Mar 2017, Gabriel F. T. Gomes wrote:

> The declarations of many functions in math_private.h are not required
> since __MATHDECL and __MATHDECLX, in math.h, already provide the
> declarations for these functions.  This patch removes the declarations
> from math_private.h. It also adds the inclusion of math.h to the files
> which depended on the declaration of functions in math_private.h.
> 
> Tested for powerpc64le and s390x.

OK apart from the removal of the inline definition of __copysignl.  That 
inline definition is still relevant in the absence of changing all calls 
to __copysign* to call copysign* directly so the compiler can inline them 
(and if changed to call copysign*, which I think would make sense, it 
would be desirable to run build-many-glibcs.py tests to make sure this 
doesn't introduce linknamespace issues).

-- 
Joseph S. Myers
joseph@codesourcery.com


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