This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] New generic cosf
On 12/05/2017 07:11 PM, Matt Turner wrote:
> On Tue, Dec 5, 2017 at 12:45 PM, 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.
>>
>> -- 8< --
>> For s390, this is the improvement noted.
>>
>> With patch:
>> "cosf": {
>> "": {
>> "duration": 1.00479e+10,
>> "iterations": 1.53856e+08,
>> "max": 900.645,
>> "min": 4.264,
>> "mean": 65.3074
>> }
>> }
>> Without patch:
>> "cosf": {
>> "": {
>> "duration": 9.93841e+09,
>> "iterations": 4.63972e+08,
>> "max": 1010.9,
>> "min": 6.593,
>> "mean": 21.4203
>> }
>> }
>
> Did I misunderstand, or did the mean time increase with the patch?
Sigh. It's not you. I was trying to mimic Raji's patch, but confused her "with/without" with what I'd normally use "before/after". I'll rectify in V2.
PC