PATCH x86_64 only: optimize sinf and cosf with SSE2
Dmitrieva Liubov
liubov.dmitrieva@gmail.com
Mon Aug 13 14:06:00 GMT 2012
I've fixed the sinf and cosf x86_64 patch according to requests below
and reattach it.
And yes, I intend to provide SSE optimized sincos in couple weeks for
the patch: http://sourceware.org/ml/libc-alpha/2012-06/msg00692.html
Change log is the same:
2012-08-13 Liubov Dmitrieva <liubov.dmitrieva@gmail.com>
* sysdeps/x86_64/fpu/s_sinf.S New file.
* sysdeps/x86_64/fpu/s_cosf.S New file.
* sysdeps/x86_64/fpu/libm-test-ulps Update.
--
Liubov Dmitrieva
Intel Corporation
2012/8/9 Markus Trippelsdorf <markus@trippelsdorf.de>:
> On 2012.08.09 at 10:39 +0200, Andreas Jaeger wrote:
>> On Thursday, August 09, 2012 12:16:06 Dmitrieva Liubov wrote:
>> > Repost of the patch with opimized SINF and COSF for x86_64.
>> >
>> > http://sourceware.org/ml/libc-alpha/2012-06/msg00692.html
>> >
>> > We are looking forward to accepting and releasing this.
>>
>> Btw. if you could look at this bugreport as well and provide an
>> optimized sincos implementation, it would be appreciated:
>> http://sourceware.org/bugzilla/show_bug.cgi?id=14412
>>
>> Just a few comments on your submission:
>>
>> * Please add a copyright header.
>> * Instead of defining EDOM, I suggest to:
>> #include <bits/errno.h>
>>
>> * For your comments, follow the usual style - with two spaces after "."
>> or "?" - and no space before
>
> Also ASM_TYPE_DIRECTIVE{,_PREFIX} was removed a few days ago (commit
> b67e9372b28ef33). Please use ".type L(FOO),@object" instead.
>
> --
> Markus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: sinf_cosf_x86_64.patch
Type: application/octet-stream
Size: 35690 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/libc-alpha/attachments/20120813/d8b625f0/attachment.obj>
More information about the Libc-alpha
mailing list