Re: [Mingw-w64-public] [PATCH 1/3] math: Call the normal double copysign in copysignl on arm

2017-08-04 Thread Martin Storsjö
On Sat, 5 Aug 2017, JonY via Mingw-w64-public wrote: On 08/05/2017 04:38 AM, Martin Storsjö wrote: On Sat, 5 Aug 2017, JonY via Mingw-w64-public wrote: On 08/04/2017 08:37 PM, Martin Storsjö wrote: We already have a C based copysign function which is used on arm. On arm, long double is equal

Re: [Mingw-w64-public] [PATCH 1/3] math: Call the normal double copysign in copysignl on arm

2017-08-04 Thread JonY via Mingw-w64-public
On 08/05/2017 04:38 AM, Martin Storsjö wrote: > On Sat, 5 Aug 2017, JonY via Mingw-w64-public wrote: > >> On 08/04/2017 08:37 PM, Martin Storsjö wrote: >>> We already have a C based copysign function which is used on arm. >>> On arm, long double is equal to double, so just call the normal >>> doub

Re: [Mingw-w64-public] [PATCH 1/3] math: Call the normal double copysign in copysignl on arm

2017-08-04 Thread Martin Storsjö
On Sat, 5 Aug 2017, JonY via Mingw-w64-public wrote: On 08/04/2017 08:37 PM, Martin Storsjö wrote: We already have a C based copysign function which is used on arm. On arm, long double is equal to double, so just call the normal double function instead of using the handwritten assembly. Chan

Re: [Mingw-w64-public] [PATCH 1/3] math: Call the normal double copysign in copysignl on arm

2017-08-04 Thread JonY via Mingw-w64-public
On 08/04/2017 08:37 PM, Martin Storsjö wrote: > We already have a C based copysign function which is used on arm. > On arm, long double is equal to double, so just call the normal > double function instead of using the handwritten assembly. > Changes in the series look good for master. signatu

[Mingw-w64-public] [PATCH 1/3] math: Call the normal double copysign in copysignl on arm

2017-08-04 Thread Martin Storsjö
We already have a C based copysign function which is used on arm. On arm, long double is equal to double, so just call the normal double function instead of using the handwritten assembly. The C based copysign function ends up at 5 instructions including "bx lr", which is significantly shorter tha