This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] s_sinf.c: Replace floor with FLOOR_DOUBLE_TO_INT/FLOOR_INT_TO_DOUBLE_HALF

From: Joseph Myers <joseph at codesourcery dot com>
To: Rajalakshmi Srinivasaraghavan <raji at linux dot vnet dot ibm dot com>
Cc: <libc-alpha at sourceware dot org>
Date: Tue, 5 Dec 2017 16:20:41 +0000
Subject: Re: [PATCH] s_sinf.c: Replace floor with FLOOR_DOUBLE_TO_INT/FLOOR_INT_TO_DOUBLE_HALF
Authentication-results: sourceware.org; auth=none
References: <20171205130832.GA3696@intel.com> <alpine.DEB.2.20.1712051318010.29414@digraph.polyomino.org.uk> <CAMe9rOqbjJNmdMUhpFN-vjNempycot-RZ_xJx4V4BKX-j-e1ow@mail.gmail.com> <8ee4069a-5dcc-8854-25e7-a462bc12c50d@linux.vnet.ibm.com>

On Tue, 5 Dec 2017, Rajalakshmi Srinivasaraghavan wrote:

> > Here is the patch to replace floor with simple casts.  It improves
> > performance
> > of max and mean by 30% on x86-64.  Rajalakshmi, can you try it on s390?
> 
> Mean value reduced from 22.0 to 20.0 with this patch on s390x.

Thanks.  The patch is OK.

-- 
Joseph S. Myers
joseph@codesourcery.com

References:
- [PATCH] s_sinf.c: Replace floor with FLOOR_DOUBLE_TO_INT/FLOOR_INT_TO_DOUBLE_HALF
  - From: H.J. Lu
- Re: [PATCH] s_sinf.c: Replace floor with FLOOR_DOUBLE_TO_INT/FLOOR_INT_TO_DOUBLE_HALF
  - From: Joseph Myers
- Re: [PATCH] s_sinf.c: Replace floor with FLOOR_DOUBLE_TO_INT/FLOOR_INT_TO_DOUBLE_HALF
  - From: H.J. Lu
- Re: [PATCH] s_sinf.c: Replace floor with FLOOR_DOUBLE_TO_INT/FLOOR_INT_TO_DOUBLE_HALF
  - From: Rajalakshmi Srinivasaraghavan

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]