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] Refactor part of math Makefile



On 06/14/2016 04:04 PM, Joseph Myers wrote:
> On Tue, 14 Jun 2016, Paul E. Murphy wrote:
> 
>> +	     gamma_productF k_standardF lgamma_negF lgamma_productF		\
>> +
>> +libm-compat-calls = w_lgamma_compatf w_lgamma_compat w_lgamma_compatl
> 
> Given the conclusion that we should obsolete matherr / _LIB_VERSION as 
> part of the preparation for adding a new type, k_standard should end up in 
> the set of compat calls as well.  (The new cleaner wrappers used for new 
> types, and for static linking / new ports for existing types, would handle 
> errno setting directly without calling __kernel_standard.)

Is there a reason not to continue calling the function __kernel_standard
and build a compat variant named __kernel_standard_compat to preserve the
legacy behavior (with similar additions to this variable).

Anyhow, that change should be done on its own.  Its a trivial change to
this patch.

> 
>> +libm-tests = $(foreach t,$(types),test-$(t) 				\
>> +				  test-$(t)-finite 			\
>> +				  test-i$(subst ldouble,ldoubl,$(t)))	\
> 
> I don't think we should keep the irregularity of the test being called 
> test-ildoubl rather than ildouble.  Rename it and you no longer need a 
> special $(subst ldouble,ldoubl,$(t)).

I'm happy to oblige there.  Do you happen to know the reason for the odd
naming?


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