On Sat, 25 Feb 2023 at 09:27, Richard Henderson <richard.hender...@linaro.org> wrote: > > Translators are no longer required to free tcg temporaries, > therefore there's no need to record temps for later freeing. > Replace the few uses with tcg_temp_new_i32. > > Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
> @@ -5762,14 +5752,7 @@ static void disas_sparc_insn(DisasContext * dc, > unsigned int insn) > gen_exception(dc, TT_NCP_INSN); > goto egress; > #endif > - egress: > - if (dc->n_t32 != 0) { > - int i; > - for (i = dc->n_t32 - 1; i >= 0; --i) { > - tcg_temp_free_i32(dc->t32[i]); > - } > - dc->n_t32 = 0; > - } > + egress:; > } Stray semicolon after colon, or does C not let you put a label at the end of a function ? Otherwise Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> thanks -- PMM