This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v3] New generic cosf
On 12/07/2017 11:59 AM, H.J. Lu wrote:
> On Thu, Dec 7, 2017 at 9:07 AM, Paul Clarke <pc@us.ibm.com> wrote:
>> The same logic used in s_cosf.S version for x86 and powerpc
>> is used to create a generic s_cosf.c, so there is no performance
>> improvement in x86_64 and powerpc64.
>>
>> 2017-12-06 Paul A. Clarke <pc@us.ibm.com>
>>
>> * sysdeps/ieee754/flt-32/s_cosf.c: New implementation.
>
> Please take a look at sysdeps/ieee754/flt-32/math_config.h to see how
> to share constants between s_cosf.c and s_sinf.c.
I interpreted Joseph's response to your earlier suggestion of the same as it to be acceptable to common these tables and constants into a future submission of a combined sinf/cosf/sincosf.
Raji is looking into that approach.
As for cosf, I figured committing a good cosf now is better than waiting for a perfect sinf/cosf/sincosf. Unless there are strong objections?
(FYI, for some reason, I'm not receiving my own posts back to my inbox, so I have been unable to reply to them. This seems like a recent change in behavior.)
PC