[PATCH 1/3] Update s_sincosf.c and x86-64 s_sincosf-fma.c

[Wilco Dijkstra]

Hi,

> H.J. Lu wrote:
> Only sincosf_poly is vectorized.  Without changing the existing
> structure, I need
> to duplicate everything in sysdeps/ieee754/flt-32/s_sincosf.h.

The only shared function which relies on the structure is reduce_fast.
To avoid any duplication it could just get the 2 values it needs as arguments.

> My x86-64 vector version has x86-64 specific intrinsics:
>
>  __v2df vps1c2 = (__v2df) _mm_loadu_pd (&p->s1c2.s1);
>  __v2df vps2c3 = (__v2df) _mm_loadu_pd (&p->s2c3.s2);
>  __v2df vps3c4 = (__v2df) _mm_loadu_pd (&p->s3c4.s3);

Simple vector loads are supported, you don't need any intrinsics for these.

>  __v4sf v4sf = _mm_cvtpd_ps (vsincos);

This works fine too for me using v4sf = {(float)vsincos[0], (float)vsincos[1]};

Wilco