motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-17 23:22:12 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

tcg: Clean up local variable shadowing

Browse source 
Fix:

  tcg/tcg.c:2551:27: error: declaration shadows a local variable [-Werror,-Wshadow]
                    MemOp op = get_memop(oi);
                          ^
  tcg/tcg.c:2437:12: note: previous declaration is here
    TCGOp *op;
           ^
  accel/tcg/tb-maint.c:245:18: error: declaration shadows a local variable [-Werror,-Wshadow]
        for (int i = 0; i < V_L2_SIZE; i++) {
                 ^
  accel/tcg/tb-maint.c:210:9: note: previous declaration is here
    int i;
        ^

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Message-ID: <20230904161235.84651-2-philmd@linaro.org>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
This commit is contained in:
Philippe Mathieu-Daudé 2023-09-04 18:12:13 +02:00 • committed by Markus Armbruster
parent bb71846325
commit 9a239c6eae
2 changed files with 9 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

3
accel/tcg/tb-maint.c
View file

@ -207,13 +207,12 @@ static PageDesc *page_find_alloc(tb_page_addr_t index, bool alloc)
{ {
PageDesc *pd; PageDesc *pd;
void **lp; void **lp;
int i;
/* Level 1. Always allocated. */ /* Level 1. Always allocated. */
lp = l1_map + ((index >> v_l1_shift) & (v_l1_size - 1)); lp = l1_map + ((index >> v_l1_shift) & (v_l1_size - 1));
/* Level 2..N-1. */ /* Level 2..N-1. */
for (i = v_l2_levels; i > 0; i--) { for (int i = v_l2_levels; i > 0; i--) {
void **p = qatomic_rcu_read(lp); void **p = qatomic_rcu_read(lp);
if (p == NULL) { if (p == NULL) {

16
tcg/tcg.c
View file

@ -2549,21 +2549,21 @@ static void tcg_dump_ops(TCGContext *s, FILE *f, bool have_prefs)
{ {
const char *s_al, *s_op, *s_at; const char *s_al, *s_op, *s_at;
MemOpIdx oi = op->args[k++]; MemOpIdx oi = op->args[k++];
MemOp op = get_memop(oi); MemOp mop = get_memop(oi);
unsigned ix = get_mmuidx(oi); unsigned ix = get_mmuidx(oi);
s_al = alignment_name[(op & MO_AMASK) >> MO_ASHIFT]; s_al = alignment_name[(mop & MO_AMASK) >> MO_ASHIFT];
s_op = ldst_name[op & (MO_BSWAP | MO_SSIZE)]; s_op = ldst_name[mop & (MO_BSWAP | MO_SSIZE)];
s_at = atom_name[(op & MO_ATOM_MASK) >> MO_ATOM_SHIFT]; s_at = atom_name[(mop & MO_ATOM_MASK) >> MO_ATOM_SHIFT];
op &= ~(MO_AMASK | MO_BSWAP | MO_SSIZE | MO_ATOM_MASK); mop &= ~(MO_AMASK | MO_BSWAP | MO_SSIZE | MO_ATOM_MASK);
/* If all fields are accounted for, print symbolically. */ /* If all fields are accounted for, print symbolically. */
if (!op && s_al && s_op && s_at) { if (!mop && s_al && s_op && s_at) {
col += ne_fprintf(f, ",%s%s%s,%u", col += ne_fprintf(f, ",%s%s%s,%u",
s_at, s_al, s_op, ix); s_at, s_al, s_op, ix);
} else { } else {
op = get_memop(oi); mop = get_memop(oi);
col += ne_fprintf(f, ",$0x%x,%u", op, ix); col += ne_fprintf(f, ",$0x%x,%u", mop, ix);
} }
i = 1; i = 1;
} }