aboutsummaryrefslogtreecommitdiff
path: root/tcg/mips
diff options
context:
space:
mode:
authorRichard Henderson <rth@twiddle.net>2014-04-15 22:20:57 -0700
committerRichard Henderson <rth@twiddle.net>2014-05-24 08:46:49 -0700
commitc068896f7f10f82d96a986ceea0d69d8579e3932 (patch)
tree0ba322e9acb3aa4cfe2d34de79021f52e4bd261c /tcg/mips
parentfd1cf66630a220991c837e53d9a958cd29444ba1 (diff)
downloadqemu-c068896f7f10f82d96a986ceea0d69d8579e3932.zip
qemu-c068896f7f10f82d96a986ceea0d69d8579e3932.tar.gz
qemu-c068896f7f10f82d96a986ceea0d69d8579e3932.tar.bz2
tcg-mips: Simplify brcond
Use the same table to fold comparisons as with setcond. Reviewed-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Richard Henderson <rth@twiddle.net>
Diffstat (limited to 'tcg/mips')
-rw-r--r--tcg/mips/tcg-target.c87
1 files changed, 41 insertions, 46 deletions
diff --git a/tcg/mips/tcg-target.c b/tcg/mips/tcg-target.c
index 278925c..c0a7a04 100644
--- a/tcg/mips/tcg-target.c
+++ b/tcg/mips/tcg-target.c
@@ -634,70 +634,65 @@ static void tcg_out_setcond(TCGContext *s, TCGCond cond, TCGReg ret,
}
}
-static void tcg_out_brcond(TCGContext *s, TCGCond cond, TCGArg arg1,
- TCGArg arg2, int label_index)
+static void tcg_out_brcond(TCGContext *s, TCGCond cond, TCGReg arg1,
+ TCGReg arg2, int label_index)
{
- TCGLabel *l = &s->labels[label_index];
+ static const MIPSInsn b_zero[16] = {
+ [TCG_COND_LT] = OPC_BLTZ,
+ [TCG_COND_GT] = OPC_BGTZ,
+ [TCG_COND_LE] = OPC_BLEZ,
+ [TCG_COND_GE] = OPC_BGEZ,
+ };
+
+ TCGLabel *l;
+ MIPSInsn s_opc = OPC_SLTU;
+ MIPSInsn b_opc;
+ int cmp_map;
switch (cond) {
case TCG_COND_EQ:
- tcg_out_opc_br(s, OPC_BEQ, arg1, arg2);
+ b_opc = OPC_BEQ;
break;
case TCG_COND_NE:
- tcg_out_opc_br(s, OPC_BNE, arg1, arg2);
+ b_opc = OPC_BNE;
break;
+
case TCG_COND_LT:
- if (arg2 == 0) {
- tcg_out_opc_br(s, OPC_BLTZ, 0, arg1);
- } else {
- tcg_out_opc_reg(s, OPC_SLT, TCG_TMP0, arg1, arg2);
- tcg_out_opc_br(s, OPC_BNE, TCG_TMP0, TCG_REG_ZERO);
- }
- break;
- case TCG_COND_LTU:
- tcg_out_opc_reg(s, OPC_SLTU, TCG_TMP0, arg1, arg2);
- tcg_out_opc_br(s, OPC_BNE, TCG_TMP0, TCG_REG_ZERO);
- break;
- case TCG_COND_GE:
- if (arg2 == 0) {
- tcg_out_opc_br(s, OPC_BGEZ, 0, arg1);
- } else {
- tcg_out_opc_reg(s, OPC_SLT, TCG_TMP0, arg1, arg2);
- tcg_out_opc_br(s, OPC_BEQ, TCG_TMP0, TCG_REG_ZERO);
- }
- break;
- case TCG_COND_GEU:
- tcg_out_opc_reg(s, OPC_SLTU, TCG_TMP0, arg1, arg2);
- tcg_out_opc_br(s, OPC_BEQ, TCG_TMP0, TCG_REG_ZERO);
- break;
+ case TCG_COND_GT:
case TCG_COND_LE:
+ case TCG_COND_GE:
if (arg2 == 0) {
- tcg_out_opc_br(s, OPC_BLEZ, 0, arg1);
- } else {
- tcg_out_opc_reg(s, OPC_SLT, TCG_TMP0, arg2, arg1);
- tcg_out_opc_br(s, OPC_BEQ, TCG_TMP0, TCG_REG_ZERO);
+ b_opc = b_zero[cond];
+ arg2 = arg1;
+ arg1 = 0;
+ break;
}
- break;
+ s_opc = OPC_SLT;
+ /* FALLTHRU */
+
+ case TCG_COND_LTU:
+ case TCG_COND_GTU:
case TCG_COND_LEU:
- tcg_out_opc_reg(s, OPC_SLTU, TCG_TMP0, arg2, arg1);
- tcg_out_opc_br(s, OPC_BEQ, TCG_TMP0, TCG_REG_ZERO);
- break;
- case TCG_COND_GT:
- if (arg2 == 0) {
- tcg_out_opc_br(s, OPC_BGTZ, 0, arg1);
- } else {
- tcg_out_opc_reg(s, OPC_SLT, TCG_TMP0, arg2, arg1);
- tcg_out_opc_br(s, OPC_BNE, TCG_TMP0, TCG_REG_ZERO);
+ case TCG_COND_GEU:
+ cmp_map = mips_cmp_map[cond];
+ if (cmp_map & MIPS_CMP_SWAP) {
+ TCGReg t = arg1;
+ arg1 = arg2;
+ arg2 = t;
}
+ tcg_out_opc_reg(s, s_opc, TCG_TMP0, arg1, arg2);
+ b_opc = (cmp_map & MIPS_CMP_INV ? OPC_BEQ : OPC_BNE);
+ arg1 = TCG_TMP0;
+ arg2 = TCG_REG_ZERO;
break;
- case TCG_COND_GTU:
- tcg_out_opc_reg(s, OPC_SLTU, TCG_TMP0, arg2, arg1);
- tcg_out_opc_br(s, OPC_BNE, TCG_TMP0, TCG_REG_ZERO);
- break;
+
default:
tcg_abort();
break;
}
+
+ tcg_out_opc_br(s, b_opc, arg1, arg2);
+ l = &s->labels[label_index];
if (l->has_value) {
reloc_pc16(s->code_ptr - 1, l->u.value_ptr);
} else {