[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 07/34] tcg: Remove check_regs
From: |
Richard Henderson |
Subject: |
[PATCH v3 07/34] tcg: Remove check_regs |
Date: |
Thu, 1 Dec 2022 21:39:31 -0800 |
We now check the consistency of reg_to_temp[] with each update,
so the utility of checking consistency at the end of each
opcode is minimal. In addition, the form of this check is
quite expensive, consuming 10% of a checking-enabled build.
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
tcg/tcg.c | 76 -------------------------------------------------------
1 file changed, 76 deletions(-)
diff --git a/tcg/tcg.c b/tcg/tcg.c
index c075c97050..cfcf959894 100644
--- a/tcg/tcg.c
+++ b/tcg/tcg.c
@@ -2917,79 +2917,6 @@ static bool liveness_pass_2(TCGContext *s)
return changes;
}
-#ifdef CONFIG_DEBUG_TCG
-static void dump_regs(TCGContext *s)
-{
- TCGTemp *ts;
- int i;
- char buf[64];
-
- for(i = 0; i < s->nb_temps; i++) {
- ts = &s->temps[i];
- printf(" %10s: ", tcg_get_arg_str_ptr(s, buf, sizeof(buf), ts));
- switch(ts->val_type) {
- case TEMP_VAL_REG:
- printf("%s", tcg_target_reg_names[ts->reg]);
- break;
- case TEMP_VAL_MEM:
- printf("%d(%s)", (int)ts->mem_offset,
- tcg_target_reg_names[ts->mem_base->reg]);
- break;
- case TEMP_VAL_CONST:
- printf("$0x%" PRIx64, ts->val);
- break;
- case TEMP_VAL_DEAD:
- printf("D");
- break;
- default:
- printf("???");
- break;
- }
- printf("\n");
- }
-
- for(i = 0; i < TCG_TARGET_NB_REGS; i++) {
- if (s->reg_to_temp[i] != NULL) {
- printf("%s: %s\n",
- tcg_target_reg_names[i],
- tcg_get_arg_str_ptr(s, buf, sizeof(buf),
s->reg_to_temp[i]));
- }
- }
-}
-
-static void check_regs(TCGContext *s)
-{
- int reg;
- int k;
- TCGTemp *ts;
- char buf[64];
-
- for (reg = 0; reg < TCG_TARGET_NB_REGS; reg++) {
- ts = s->reg_to_temp[reg];
- if (ts != NULL) {
- if (ts->val_type != TEMP_VAL_REG || ts->reg != reg) {
- printf("Inconsistency for register %s:\n",
- tcg_target_reg_names[reg]);
- goto fail;
- }
- }
- }
- for (k = 0; k < s->nb_temps; k++) {
- ts = &s->temps[k];
- if (ts->val_type == TEMP_VAL_REG
- && ts->kind != TEMP_FIXED
- && s->reg_to_temp[ts->reg] != ts) {
- printf("Inconsistency for temp %s:\n",
- tcg_get_arg_str_ptr(s, buf, sizeof(buf), ts));
- fail:
- printf("reg state:\n");
- dump_regs(s);
- tcg_abort();
- }
- }
-}
-#endif
-
static void temp_allocate_frame(TCGContext *s, TCGTemp *ts)
{
intptr_t off, size, align;
@@ -4328,9 +4255,6 @@ int tcg_gen_code(TCGContext *s, TranslationBlock *tb,
target_ulong pc_start)
tcg_reg_alloc_op(s, op);
break;
}
-#ifdef CONFIG_DEBUG_TCG
- check_regs(s);
-#endif
/* Test for (pending) buffer overflow. The assumption is that any
one operation beginning below the high water mark cannot overrun
the buffer completely. Thus we can test for overflow after
--
2.34.1
- [PATCH for-8.0 v3 00/34] tcg misc patches, Richard Henderson, 2022/12/02
- [PATCH v3 02/34] meson: Move CONFIG_TCG_INTERPRETER to config_host, Richard Henderson, 2022/12/02
- [PATCH v3 03/34] tcg/s390x: Fix coding style, Richard Henderson, 2022/12/02
- [PATCH v3 04/34] tcg: Cleanup trailing whitespace, Richard Henderson, 2022/12/02
- [PATCH v3 05/34] tcg: Fix tcg_reg_alloc_dup*, Richard Henderson, 2022/12/02
- [PATCH v3 06/34] tcg: Centralize updates to reg_to_temp, Richard Henderson, 2022/12/02
- [PATCH v3 01/34] tcg: convert tcg/README to rst, Richard Henderson, 2022/12/02
- [PATCH v3 07/34] tcg: Remove check_regs,
Richard Henderson <=
- [PATCH v3 10/34] tcg: Remove TCG_TARGET_STACK_GROWSUP, Richard Henderson, 2022/12/02
- [PATCH v3 09/34] tcg: Introduce paired register allocation, Richard Henderson, 2022/12/02
- [PATCH v3 11/34] accel/tcg: Set cflags_next_tb in cpu_common_initfn, Richard Henderson, 2022/12/02
- [PATCH v3 08/34] tcg: Tidy tcg_reg_alloc_op, Richard Henderson, 2022/12/02
- [PATCH v3 12/34] target/sparc: Avoid TCGV_{LOW,HIGH}, Richard Henderson, 2022/12/02
- [PATCH v3 13/34] tcg: Move TCG_{LOW,HIGH} to tcg-internal.h, Richard Henderson, 2022/12/02
- [PATCH v3 14/34] tcg: Add temp_subindex to TCGTemp, Richard Henderson, 2022/12/02
- [PATCH v3 15/34] tcg: Simplify calls to temp_sync vs mem_coherent, Richard Henderson, 2022/12/02
- [PATCH v3 17/34] tcg: Move TCG_TYPE_COUNT outside enum, Richard Henderson, 2022/12/02
- [PATCH v3 16/34] tcg: Allocate TCGTemp pairs in host memory order, Richard Henderson, 2022/12/02