This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] New generic cosf
- From: Matt Turner <mattst88 at gmail dot com>
- To: Paul Clarke <pc at us dot ibm dot com>
- Cc: GNU C Library <libc-alpha at sourceware dot org>
- Date: Tue, 5 Dec 2017 17:11:16 -0800
- Subject: Re: [PATCH] New generic cosf
- Authentication-results: sourceware.org; auth=none
- References: <72875dc7-0d21-17e6-3802-4cca20b62371@us.ibm.com>
On Tue, Dec 5, 2017 at 12:45 PM, Paul Clarke <pc@us.ibm.com> wrote:
> From 60e12f38145e4699cf231f7815cf1894d5e2d68d Mon Sep 17 00:00:00 2001
> From: "Paul A. Clarke" <pc@us.ibm.com>
> Date: Tue, 5 Dec 2017 09:32:56 -0600
> Subject: [PATCH] New generic cosf
>
> 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?