On Mon, 4 Sept 2023 at 17:12, Philippe Mathieu-Daudé <phi...@linaro.org> wrote: > > Fix: > > target/arm/tcg/translate-m-nocp.c: In function ‘gen_M_fp_sysreg_read’: > target/arm/tcg/translate-m-nocp.c:509:18: warning: declaration of ‘tmp’ > shadows a previous local [-Wshadow=compatible-local] > 509 | TCGv_i32 tmp = load_cpu_field(v7m.fpdscr[M_REG_NS]); > | ^~~ > target/arm/tcg/translate-m-nocp.c:433:14: note: shadowed declaration is here > 433 | TCGv_i32 tmp; > | ^~~ > --- > > target/arm/tcg/mve_helper.c: In function ‘helper_mve_vqshlsb’: > target/arm/tcg/mve_helper.c:1259:19: warning: declaration of ‘r’ shadows a > previous local [-Wshadow=compatible-local] > 1259 | typeof(N) r = FN(N, (int8_t)(M), sizeof(N) * 8, ROUND, > &su32); \ > | ^ > target/arm/tcg/mve_helper.c:1267:5: note: in expansion of macro > ‘WRAP_QRSHL_HELPER’ > 1267 | WRAP_QRSHL_HELPER(do_sqrshl_bhs, N, M, false, satp) > | ^~~~~~~~~~~~~~~~~ > target/arm/tcg/mve_helper.c:927:22: note: in expansion of macro > ‘DO_SQSHL_OP’ > 927 | TYPE r = FN(n[H##ESIZE(e)], m[H##ESIZE(e)], &sat); > \ > | ^~ > target/arm/tcg/mve_helper.c:945:5: note: in expansion of macro ‘DO_2OP_SAT’ > 945 | DO_2OP_SAT(OP##b, 1, int8_t, FN) \ > | ^~~~~~~~~~ > target/arm/tcg/mve_helper.c:1277:1: note: in expansion of macro > ‘DO_2OP_SAT_S’ > 1277 | DO_2OP_SAT_S(vqshls, DO_SQSHL_OP) > | ^~~~~~~~~~~~ > --- > > target/arm/tcg/mve_helper.c: In function ‘do_sqrshl48_d’: > target/arm/tcg/mve_helper.c:2463:17: warning: declaration of ‘extval’ > shadows a previous local [-Wshadow=compatible-local] > 2463 | int64_t extval = sextract64(src << shift, 0, 48); > | ^~~~~~ > target/arm/tcg/mve_helper.c:2443:18: note: shadowed declaration is here > 2443 | int64_t val, extval; > | ^~~~~~ > --- > > target/arm/tcg/mve_helper.c: In function ‘do_uqrshl48_d’: > target/arm/tcg/mve_helper.c:2495:18: warning: declaration of ‘extval’ > shadows a previous local [-Wshadow=compatible-local] > 2495 | uint64_t extval = extract64(src << shift, 0, 48); > | ^~~~~~ > target/arm/tcg/mve_helper.c:2479:19: note: shadowed declaration is here > 2479 | uint64_t val, extval; > | ^~~~~~ > > Signed-off-by: Philippe Mathieu-Daudé <phi...@linaro.org>
Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> thanks -- PMM