From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Kbcc7-00082Y-W3 for qemu-devel@nongnu.org; Fri, 05 Sep 2008 10:48:44 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Kbcc5-00080l-Rs for qemu-devel@nongnu.org; Fri, 05 Sep 2008 10:48:41 -0400 Received: from [199.232.76.173] (port=48059 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Kbcc4-00080H-V6 for qemu-devel@nongnu.org; Fri, 05 Sep 2008 10:48:41 -0400 Received: from savannah.gnu.org ([199.232.41.3]:45689 helo=sv.gnu.org) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1Kbc9g-0007Gk-Iu for qemu-devel@nongnu.org; Fri, 05 Sep 2008 10:19:20 -0400 Received: from cvs.savannah.gnu.org ([199.232.41.69]) by sv.gnu.org with esmtp (Exim 4.63) (envelope-from ) id 1Kbc9e-0007vz-Tr for qemu-devel@nongnu.org; Fri, 05 Sep 2008 14:19:19 +0000 Received: from aurel32 by cvs.savannah.gnu.org with local (Exim 4.63) (envelope-from ) id 1Kbc9e-0007vu-Ef for qemu-devel@nongnu.org; Fri, 05 Sep 2008 14:19:18 +0000 MIME-Version: 1.0 Errors-To: aurel32 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit From: Aurelien Jarno Message-Id: Date: Fri, 05 Sep 2008 14:19:18 +0000 Subject: [Qemu-devel] [5164] TCG fixes for target-mips Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Revision: 5164 http://svn.sv.gnu.org/viewvc/?view=rev&root=qemu&revision=5164 Author: aurel32 Date: 2008-09-05 14:19:17 +0000 (Fri, 05 Sep 2008) Log Message: ----------- TCG fixes for target-mips This patch fixes TCG errors reported on the MIPS target when TCG_DEBUG is enabled. Signed-off-by: Aurelien Jarno Acked-by: Thiemo Seufer Modified Paths: -------------- trunk/target-mips/translate.c Modified: trunk/target-mips/translate.c =================================================================== --- trunk/target-mips/translate.c 2008-09-05 12:33:31 UTC (rev 5163) +++ trunk/target-mips/translate.c 2008-09-05 14:19:17 UTC (rev 5164) @@ -427,7 +427,8 @@ #include "gen-icount.h" -static inline void tcg_gen_helper_0_i(void *func, TCGv arg) +static inline void tcg_gen_helper_0_i(void *func, uint32_t arg) + { TCGv tmp = tcg_const_i32(arg); @@ -435,7 +436,7 @@ tcg_temp_free(tmp); } -static inline void tcg_gen_helper_0_ii(void *func, TCGv arg1, TCGv arg2) +static inline void tcg_gen_helper_0_ii(void *func, uint32_t arg1, uint32_t arg2) { TCGv tmp1 = tcg_const_i32(arg1); TCGv tmp2 = tcg_const_i32(arg2); @@ -445,7 +446,7 @@ tcg_temp_free(tmp2); } -static inline void tcg_gen_helper_0_1i(void *func, TCGv arg1, TCGv arg2) +static inline void tcg_gen_helper_0_1i(void *func, TCGv arg1, uint32_t arg2) { TCGv tmp = tcg_const_i32(arg2); @@ -453,7 +454,7 @@ tcg_temp_free(tmp); } -static inline void tcg_gen_helper_0_2i(void *func, TCGv arg1, TCGv arg2, TCGv arg3) +static inline void tcg_gen_helper_0_2i(void *func, TCGv arg1, TCGv arg2, uint32_t arg3) { TCGv tmp = tcg_const_i32(arg3); @@ -461,7 +462,7 @@ tcg_temp_free(tmp); } -static inline void tcg_gen_helper_0_1ii(void *func, TCGv arg1, TCGv arg2, TCGv arg3) +static inline void tcg_gen_helper_0_1ii(void *func, TCGv arg1, uint32_t arg2, uint32_t arg3) { TCGv tmp1 = tcg_const_i32(arg2); TCGv tmp2 = tcg_const_i32(arg3); @@ -471,7 +472,7 @@ tcg_temp_free(tmp2); } -static inline void tcg_gen_helper_1_i(void *func, TCGv ret, TCGv arg) +static inline void tcg_gen_helper_1_i(void *func, TCGv ret, uint32_t arg) { TCGv tmp = tcg_const_i32(arg); @@ -479,7 +480,7 @@ tcg_temp_free(tmp); } -static inline void tcg_gen_helper_1_1i(void *func, TCGv ret, TCGv arg1, TCGv arg2) +static inline void tcg_gen_helper_1_1i(void *func, TCGv ret, TCGv arg1, uint32_t arg2) { TCGv tmp = tcg_const_i32(arg2); @@ -487,7 +488,7 @@ tcg_temp_free(tmp); } -static inline void tcg_gen_helper_1_1ii(void *func, TCGv ret, TCGv arg1, TCGv arg2, TCGv arg3) +static inline void tcg_gen_helper_1_1ii(void *func, TCGv ret, TCGv arg1, uint32_t arg2, uint32_t arg3) { TCGv tmp1 = tcg_const_i32(arg2); TCGv tmp2 = tcg_const_i32(arg3); @@ -497,7 +498,7 @@ tcg_temp_free(tmp2); } -static inline void tcg_gen_helper_1_2i(void *func, TCGv ret, TCGv arg1, TCGv arg2, TCGv arg3) +static inline void tcg_gen_helper_1_2i(void *func, TCGv ret, TCGv arg1, TCGv arg2, uint32_t arg3) { TCGv tmp = tcg_const_i32(arg3); @@ -505,7 +506,7 @@ tcg_temp_free(tmp); } -static inline void tcg_gen_helper_1_2ii(void *func, TCGv ret, TCGv arg1, TCGv arg2, TCGv arg3, TCGv arg4) +static inline void tcg_gen_helper_1_2ii(void *func, TCGv ret, TCGv arg1, TCGv arg2, uint32_t arg3, uint32_t arg4) { TCGv tmp1 = tcg_const_i32(arg3); TCGv tmp2 = tcg_const_i32(arg4); @@ -722,7 +723,7 @@ typedef void (fcmp_fun32)(uint32_t, uint32_t, int); typedef void (fcmp_fun64)(uint64_t, uint64_t, int); -#define FOP_CONDS(fcmp_fun, type, arg0, arg1, fmt) \ +#define FOP_CONDS(fcmp_fun, type, fmt) \ static fcmp_fun * fcmp ## type ## _ ## fmt ## _table[16] = { \ do_cmp ## type ## _ ## fmt ## _f, \ do_cmp ## type ## _ ## fmt ## _un, \ @@ -741,17 +742,17 @@ do_cmp ## type ## _ ## fmt ## _le, \ do_cmp ## type ## _ ## fmt ## _ngt, \ }; \ -static inline void gen_cmp ## type ## _ ## fmt(int n, arg0 a, arg1 b, int cc) \ +static inline void gen_cmp ## type ## _ ## fmt(int n, TCGv a, TCGv b, int cc) \ { \ tcg_gen_helper_0_2i(fcmp ## type ## _ ## fmt ## _table[n], a, b, cc); \ } -FOP_CONDS(fcmp_fun64, , uint64_t, uint64_t, d) -FOP_CONDS(fcmp_fun64, abs, uint64_t, uint64_t, d) -FOP_CONDS(fcmp_fun32, , uint32_t, uint32_t, s) -FOP_CONDS(fcmp_fun32, abs, uint32_t, uint32_t, s) -FOP_CONDS(fcmp_fun64, , uint64_t, uint64_t, ps) -FOP_CONDS(fcmp_fun64, abs, uint64_t, uint64_t, ps) +FOP_CONDS(fcmp_fun64, , d) +FOP_CONDS(fcmp_fun64, abs, d) +FOP_CONDS(fcmp_fun32, , s) +FOP_CONDS(fcmp_fun32, abs, s) +FOP_CONDS(fcmp_fun64, , ps) +FOP_CONDS(fcmp_fun64, abs, ps) #undef FOP_CONDS /* Tests */ @@ -2498,7 +2499,7 @@ { target_ulong btgt = -1; int blink = 0; - int bcond = 0; + int bcond_compute = 0; TCGv t0 = tcg_temp_local_new(TCG_TYPE_TL); TCGv t1 = tcg_temp_local_new(TCG_TYPE_TL); @@ -2524,7 +2525,7 @@ if (rs != rt) { gen_load_gpr(t0, rs); gen_load_gpr(t1, rt); - bcond = 1; + bcond_compute = 1; } btgt = ctx->pc + 4 + offset; break; @@ -2543,7 +2544,7 @@ /* Compare to zero */ if (rs != 0) { gen_load_gpr(t0, rs); - bcond = 1; + bcond_compute = 1; } btgt = ctx->pc + 4 + offset; break; @@ -2569,7 +2570,7 @@ generate_exception(ctx, EXCP_RI); goto out; } - if (bcond == 0) { + if (bcond_compute == 0) { /* No condition to be computed */ switch (opc) { case OPC_BEQ: /* rx == rx */ @@ -6045,7 +6046,7 @@ tcg_gen_ld_i32(r_tmp1, current_fpu, offsetof(CPUMIPSFPUContext, fcr31)); tcg_gen_andi_i32(r_tmp1, r_tmp1, ccbit); tcg_gen_brcondi_i32(cond, r_tmp1, 0, l1); - tcg_gen_movi_i32(fp1, fp0); + tcg_gen_mov_i32(fp1, fp0); tcg_temp_free(fp0); gen_set_label(l1); tcg_temp_free(r_tmp1); @@ -6077,7 +6078,7 @@ tcg_gen_ld_i32(r_tmp1, current_fpu, offsetof(CPUMIPSFPUContext, fcr31)); tcg_gen_andi_i32(r_tmp1, r_tmp1, ccbit); tcg_gen_brcondi_i32(cond, r_tmp1, 0, l1); - tcg_gen_movi_i64(fp1, fp0); + tcg_gen_mov_i64(fp1, fp0); tcg_temp_free(fp0); gen_set_label(l1); tcg_temp_free(r_tmp1); @@ -6110,12 +6111,12 @@ tcg_gen_shri_i32(r_tmp1, r_tmp1, cc); tcg_gen_andi_i32(r_tmp2, r_tmp1, 0x1); tcg_gen_brcondi_i32(cond, r_tmp2, 0, l1); - tcg_gen_movi_i32(fp1, fp0); + tcg_gen_mov_i32(fp1, fp0); tcg_temp_free(fp0); gen_set_label(l1); tcg_gen_andi_i32(r_tmp2, r_tmp1, 0x2); tcg_gen_brcondi_i32(cond, r_tmp2, 0, l2); - tcg_gen_movi_i32(fph1, fph0); + tcg_gen_mov_i32(fph1, fph0); tcg_temp_free(fph0); gen_set_label(l2); tcg_temp_free(r_tmp1);