From owner-cvs-all@FreeBSD.ORG  Fri Feb 22 18:43:24 2008
Return-Path: <owner-cvs-all@FreeBSD.ORG>
Delivered-To: cvs-all@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id 5EF7816A404;
	Fri, 22 Feb 2008 18:43:24 +0000 (UTC) (envelope-from bde@FreeBSD.org)
Received: from repoman.freebsd.org (repoman.freebsd.org
	[IPv6:2001:4f8:fff6::29])
	by mx1.freebsd.org (Postfix) with ESMTP id 59D5B13C4E8;
	Fri, 22 Feb 2008 18:43:24 +0000 (UTC) (envelope-from bde@FreeBSD.org)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.14.1/8.14.1) with ESMTP id m1MIhOue042214;
	Fri, 22 Feb 2008 18:43:24 GMT (envelope-from bde@repoman.freebsd.org)
Received: (from bde@localhost)
	by repoman.freebsd.org (8.14.1/8.14.1/Submit) id m1MIhOcn042213;
	Fri, 22 Feb 2008 18:43:24 GMT (envelope-from bde)
Message-Id: <200802221843.m1MIhOcn042213@repoman.freebsd.org>
From: Bruce Evans <bde@FreeBSD.org>
Date: Fri, 22 Feb 2008 18:43:23 +0000 (UTC)
To: src-committers@FreeBSD.org, cvs-src@FreeBSD.org, cvs-all@FreeBSD.org
X-FreeBSD-CVS-Branch: HEAD
Cc: 
Subject: cvs commit: src/lib/msun/src e_rem_pio2.c e_rem_pio2f.c
X-BeenThere: cvs-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: CVS commit messages for the entire tree <cvs-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-all>
List-Post: <mailto:cvs-all@freebsd.org>
List-Help: <mailto:cvs-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-all>,
	<mailto:cvs-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Fri, 22 Feb 2008 18:43:24 -0000

bde         2008-02-22 18:43:23 UTC

  FreeBSD src repository

  Modified files:
    lib/msun/src         e_rem_pio2.c e_rem_pio2f.c 
  Log:
  Avoid using FP-to-integer conversion for !(amd64 || i386) too.  Use the
  FP-to-FP method to round to an integer on all arches, and convert this
  to an int using FP-to-integer conversion iff irint() is not available.
  This is cleaner and works well on at least ia64, where it saves 20-30
  cycles or about 10% on average for 9Pi/4 < |x| <= 32pi/2 (should be
  similar up to 2**19pi/2, but I only tested the smaller range).
  
  After the previous commit to e_rem_pio2.c removed the "quick check no
  cancellation" non-optimization, the result of the FP-to-integer
  conversion is not needed so early, so using irint() became a much
  smaller optimization than when it was committed.
  
  An earlier commit message said that cos, cosf, sin and sinf were equally
  fast on amd64 and i386 except for cos and sin on i386.  Actually, cos
  and sin on amd64 are equally fast to cosf and sinf on i386 (~88 cycles),
  while cosf and sinf on amd64 are not quite equally slow to cos and sin
  on i386 (average 115 cycles with more variance).
  
  Revision  Changes    Path
  1.14      +2 -3      src/lib/msun/src/e_rem_pio2.c
  1.23      +2 -3      src/lib/msun/src/e_rem_pio2f.c