From owner-cvs-all@FreeBSD.ORG Thu Aug 17 02:40:57 2006 Return-Path: X-Original-To: cvs-all@FreeBSD.org Delivered-To: cvs-all@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id C26F516A4DA; Thu, 17 Aug 2006 02:40:57 +0000 (UTC) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.FreeBSD.org (Postfix) with ESMTP id 296AE43D49; Thu, 17 Aug 2006 02:40:56 +0000 (GMT) (envelope-from scottl@samsco.org) Received: from [192.168.254.11] (phobos.samsco.home [192.168.254.11]) (authenticated bits=0) by pooker.samsco.org (8.13.4/8.13.4) with ESMTP id k7H2emTQ036491; Wed, 16 Aug 2006 20:40:54 -0600 (MDT) (envelope-from scottl@samsco.org) Message-ID: <44E3D72D.3070604@samsco.org> Date: Wed, 16 Aug 2006 20:40:45 -0600 From: Scott Long User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.13) Gecko/20060414 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Bruce Evans References: <200608151245.k7FCjpJo077372@repoman.freebsd.org> <200608152134.46359.davidxu@freebsd.org> <20060815135517.GB41562@deviant.kiev.zoral.com.ua> <200608151052.49421.jhb@freebsd.org> <20060817003125.I38538@delplex.bde.org> In-Reply-To: <20060817003125.I38538@delplex.bde.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-1.4 required=3.8 tests=ALL_TRUSTED autolearn=failed version=3.1.1 X-Spam-Checker-Version: SpamAssassin 3.1.1 (2006-03-10) on pooker.samsco.org Cc: src-committers@FreeBSD.org, John Baldwin , cvs-src@FreeBSD.org, cvs-all@FreeBSD.org, David Xu , Kostik Belousov Subject: Re: cvs commit: src/sys/amd64/amd64 support.S X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 17 Aug 2006 02:40:57 -0000 Bruce Evans wrote: > On Tue, 15 Aug 2006, John Baldwin wrote: > >> On Tuesday 15 August 2006 09:55, Kostik Belousov wrote: >> >>> So, it seems that xorq %rax, %rax and xorl %eax, %eax will make the >>> same results, but in the different ways. And xorq requires REX prefix, >>> that shall make the decoding longer. >> >> >> Ok, thanks! David, can you revert your change? I had almost gone >> through >> earlier and fixed all the places that did this a while back until I >> realized >> that it must have been intentional. > > > It is logged as intentional in the commit that went through and optimized > all places that used xorq: > > %%% > RCS file: /home/ncvs/src/sys/amd64/amd64/support.S,v > Working file: support.S > head: 1.118 > ... > ---------------------------- > revision 1.116 > date: 2005/09/27 18:32:46; author: peter; state: Exp; lines: +11 -11 > Fix a minor nit that has been bugging me for a while. Fix the obvious > cases of using a 64 bit operation to zero a register. 32 bit opcodes are > smaller and supposedly faster, and clear the upper 32 bits for free. > ---------------------------- > %%% > > Bruce I really, truly, am constantly surprised at the number of people who 'use' CVS, or any version control system for that matter, but don't look at annotations or revision history when they are making changes. I have often had to remind co-workers that version control software is more than just a fancy/expensive FTP server. Scott