[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [5949] target-ppc: memory load/store rework
|
From: |
Aurelien Jarno |
|
Subject: |
Re: [Qemu-devel] [5949] target-ppc: memory load/store rework |
|
Date: |
Sat, 13 Dec 2008 12:15:20 +0100 |
|
User-agent: |
Mutt/1.5.18 (2008-05-17) |
On Sat, Dec 13, 2008 at 01:18:03AM +0000, Stuart Brady wrote:
> On Mon, Dec 08, 2008 at 06:11:22PM +0000, Aurelien Jarno wrote:
> > - Don't swap values twice for bit-reverse load/store functions
> > in little endian mode.
>
> If I've read this correctly, it seems to me that we would still swap
> twice on little endian hosts when when emulating little endian mode or
> performing bit-reverse loads/stores. Bit-reverse loads/stores under
> little endian emulation on a little endian host would have actually
> resulted in *three* byteswaps!
I was actually speaking about swapping in the TCG code. The global
byteswapping has been actually reduced from 3 to 2 in the worst case.
--
.''`. Aurelien Jarno | GPG: 1024D/F1BCDB73
: :' : Debian developer | Electrical Engineer
`. `' address@hidden | address@hidden
`- people.debian.org/~aurel32 | www.aurel32.net