From 0282ff4d61e6b1583108e6b7ac36ff7cebc8bd5a Mon Sep 17 00:00:00 2001 From: Aurelien Jarno Date: Mon, 27 Dec 2010 19:54:49 +0100 Subject: [PATCH] target-arm: fix bug in translation of REVSH The translation of REVSH shifted the low byte 8 steps left before performing an 8-bit sign extend, causing this part of the expression to alwas be 0. Reported-by: Johan Bengtsson Signed-off-by: Aurelien Jarno (cherry picked from commit 1a855029af40df40144a322bba0e1e61c68eed2a) Signed-off-by: Aurelien Jarno --- target-arm/translate.c | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/target-arm/translate.c b/target-arm/translate.c index 6fcdd7e544..cb19dd7b7a 100644 --- a/target-arm/translate.c +++ b/target-arm/translate.c @@ -250,13 +250,9 @@ static void gen_rev16(TCGv var) /* Byteswap low halfword and sign extend. */ static void gen_revsh(TCGv var) { - TCGv tmp = new_tmp(); - tcg_gen_shri_i32(tmp, var, 8); - tcg_gen_andi_i32(tmp, tmp, 0x00ff); - tcg_gen_shli_i32(var, var, 8); - tcg_gen_ext8s_i32(var, var); - tcg_gen_or_i32(var, var, tmp); - dead_tmp(tmp); + tcg_gen_ext16u_i32(var, var); + tcg_gen_bswap16_i32(var, var); + tcg_gen_ext16s_i32(var, var); } /* Unsigned bitfield extract. */