cvs commit: src/sys/amd64/amd64 support.S
Bruce Evans
bde at zeta.org.au
Wed Aug 16 14:34:36 UTC 2006
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
More information about the cvs-src
mailing list