Skip to content

Commit

Permalink
tcg/tci: Use g_assert_not_reached
Browse files Browse the repository at this point in the history
Three TODO instances are never happen cases.
Other uses of tcg_abort are also indicating unreachable cases.

Tested-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Stefan Weil <sw@weilnetz.de>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
  • Loading branch information
rth7680 committed Feb 5, 2021
1 parent b4d5bf0 commit f6996f9
Showing 1 changed file with 7 additions and 8 deletions.
15 changes: 7 additions & 8 deletions tcg/tci.c
Expand Up @@ -362,7 +362,7 @@ static bool tci_compare32(uint32_t u0, uint32_t u1, TCGCond condition)
result = (u0 > u1);
break;
default:
TODO();
g_assert_not_reached();
}
return result;
}
Expand Down Expand Up @@ -404,7 +404,7 @@ static bool tci_compare64(uint64_t u0, uint64_t u1, TCGCond condition)
result = (u0 > u1);
break;
default:
TODO();
g_assert_not_reached();
}
return result;
}
Expand Down Expand Up @@ -1114,7 +1114,7 @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env,
tmp32 = qemu_ld_beul;
break;
default:
tcg_abort();
g_assert_not_reached();
}
tci_write_reg(regs, t0, tmp32);
break;
Expand Down Expand Up @@ -1163,7 +1163,7 @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env,
tmp64 = qemu_ld_beq;
break;
default:
tcg_abort();
g_assert_not_reached();
}
tci_write_reg(regs, t0, tmp64);
if (TCG_TARGET_REG_BITS == 32) {
Expand Down Expand Up @@ -1191,7 +1191,7 @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env,
qemu_st_bel(t0);
break;
default:
tcg_abort();
g_assert_not_reached();
}
break;
case INDEX_op_qemu_st_i64:
Expand Down Expand Up @@ -1221,16 +1221,15 @@ uintptr_t QEMU_DISABLE_CFI tcg_qemu_tb_exec(CPUArchState *env,
qemu_st_beq(tmp64);
break;
default:
tcg_abort();
g_assert_not_reached();
}
break;
case INDEX_op_mb:
/* Ensure ordering for all kinds */
smp_mb();
break;
default:
TODO();
break;
g_assert_not_reached();
}
tci_assert(tb_ptr == old_code_ptr + op_size);
}
Expand Down

0 comments on commit f6996f9

Please sign in to comment.