From owner-cvs-all@FreeBSD.ORG  Fri Feb 22 17:26: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 7389716A400;
	Fri, 22 Feb 2008 17:26: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 6E3CE13C459;
	Fri, 22 Feb 2008 17:26: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 m1MHQOJo033514;
	Fri, 22 Feb 2008 17:26:24 GMT (envelope-from bde@repoman.freebsd.org)
Received: (from bde@localhost)
	by repoman.freebsd.org (8.14.1/8.14.1/Submit) id m1MHQOC3033513;
	Fri, 22 Feb 2008 17:26:24 GMT (envelope-from bde)
Message-Id: <200802221726.m1MHQOC3033513@repoman.freebsd.org>
From: Bruce Evans <bde@FreeBSD.org>
Date: Fri, 22 Feb 2008 17:26:24 +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
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 17:26:24 -0000

bde         2008-02-22 17:26:24 UTC

  FreeBSD src repository

  Modified files:
    lib/msun/src         e_rem_pio2.c 
  Log:
  Remove the "quick check no cancellation" optimization for
  9pi/2 < |x| < 32pi/2 since it is only a small or negative optimation
  and it gets in the way of further optimizations.  It did one more
  branch to avoid some integer operations and to use a different
  dependency on previous results.  The branches are fairly predictable
  so they are usually not a problem, so whether this is a good
  optimization depends mainly on the timing for the previous results,
  which is very machine-dependent.  On amd64 (A64), this "optimization"
  is a pessimization of about 1 cycle or 1%; on ia64, it is an
  optimization of about 2 cycles or 1%; on i386 (A64), it is an
  optimization of about 5 cycles or 4%; on i386 (Celeron P2) it is an
  optimization of about 4 cycles or 3% for cos but a pessimization of
  about 5 cycles for sin and 1 cycle for tan.  I think the new i386
  (A64) slowness is due to an pipeline stall due to an avoidable
  load-store mismatch (so the old timing was better), and the i386
  (Celeron) variance is due to its branch predictor not being too good.
  
  Revision  Changes    Path
  1.13      +1 -12     src/lib/msun/src/e_rem_pio2.c