From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56401) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WXbfJ-0006jS-V0 for qemu-devel@nongnu.org; Tue, 08 Apr 2014 15:26:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WXbf6-00065n-0I for qemu-devel@nongnu.org; Tue, 08 Apr 2014 15:26:37 -0400 From: Tom Musta Date: Tue, 8 Apr 2014 14:26:13 -0500 Message-Id: <1396985174-13212-2-git-send-email-tommusta@gmail.com> In-Reply-To: <1396985174-13212-1-git-send-email-tommusta@gmail.com> References: <1396985174-13212-1-git-send-email-tommusta@gmail.com> Subject: [Qemu-devel] [V3 PATCH 1/2] target-ppc: Fix target_disas List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Tom Musta , qemu-ppc@nongnu.org Inspect only bit 16 for the Little Endian test. Correct comment preceding the target_disas() function. Signed-off-by: Tom Musta --- V2: new disas.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/disas.c b/disas.c index 79e6944..a427d18 100644 --- a/disas.c +++ b/disas.c @@ -191,7 +191,8 @@ static int print_insn_od_target(bfd_vma pc, disassemble_info *info) values: i386 - 1 means 16 bit code, 2 means 64 bit code arm - bit 0 = thumb, bit 1 = reverse endian, bit 2 = A64 - ppc - nonzero means little endian + ppc - bits 0:15 specify (optionally) the machine instruction set; + bit 16 indicates little endian. other targets - unused */ void target_disas(FILE *out, CPUArchState *env, target_ulong code, @@ -251,7 +252,7 @@ void target_disas(FILE *out, CPUArchState *env, target_ulong code, s.info.mach = bfd_mach_sparc_v9b; #endif #elif defined(TARGET_PPC) - if (flags >> 16) { + if ((flags >> 16) & 1) { s.info.endian = BFD_ENDIAN_LITTLE; } if (flags & 0xFFFF) { -- 1.7.1