[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-ppc] [PATCH 16/16] target-ppc: Convert to new lds
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [Qemu-ppc] [PATCH 16/16] target-ppc: Convert to new ldst opcodes |
Date: |
Thu, 5 Sep 2013 11:08:24 +0200 |
On 04.09.2013, at 23:05, Richard Henderson wrote:
> This lets us change "le_mode" to "end_mode" and fold away nearly all
> of the tests for the current cpu endianness, and removing all of the
> explicitly generated bswap opcodes.
>
> Cc: address@hidden
> Signed-off-by: Richard Henderson <address@hidden>
No complaints from me, apart from the usual "LE mode isn't necessarily what you
think it is on PPC" one. But the code would be as broken as before IIUC.
Ben, you had some insight in how LE mode on different PPC flavors work. Could
you please make sure we're not walking into the wrong direction here?
Alex
- [Qemu-devel] [PATCH 07/16] tcg-ppc64: Use TCGMemOp within qemu_ldst routines, (continued)
- [Qemu-devel] [PATCH 07/16] tcg-ppc64: Use TCGMemOp within qemu_ldst routines, Richard Henderson, 2013/09/04
- [Qemu-devel] [PATCH 08/16] tcg-hppa: Use TCGMemOp within qemu_ldst routines, Richard Henderson, 2013/09/04
- [Qemu-devel] [PATCH 09/16] tcg-mips: Use TCGMemOp within qemu_ldst routines, Richard Henderson, 2013/09/04
- [Qemu-devel] [PATCH 10/16] tcg-sparc: Use TCGMemOp within qemu_ldst routines, Richard Henderson, 2013/09/04
- [Qemu-devel] [PATCH 11/16] tcg: Add qemu_ld_st_i32/64, Richard Henderson, 2013/09/04
- [Qemu-devel] [PATCH 12/16] exec: Add both big- and little-endian memory helpers, Richard Henderson, 2013/09/04
- [Qemu-devel] [PATCH 14/16] tcg-i386: Remove "cb" output restriction from qemu_st8 for i386, Richard Henderson, 2013/09/04
- [Qemu-devel] [PATCH 13/16] tcg-i386: Tidy softmmu routines, Richard Henderson, 2013/09/04
- [Qemu-devel] [PATCH 15/16] tcg-i386: Support new ldst opcodes, Richard Henderson, 2013/09/04
- [Qemu-devel] [PATCH 16/16] target-ppc: Convert to new ldst opcodes, Richard Henderson, 2013/09/04