[PATCH] Refactor part of math Makefile
Paul E. Murphy
murphyp@linux.vnet.ibm.com
Tue Jun 14 21:27:00 GMT 2016
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?
More information about the Libc-alpha
mailing list