On Fri, Jul 14, 2017 at 23:42:39 -1000, Richard Henderson wrote: > From: Lluís Vilanova <vilan...@ac.upc.edu> > > Signed-off-by: Lluís Vilanova <vilan...@ac.upc.edu> > Message-Id: <150002631325.22386.10348327185029496649.st...@frigg.lan> > Signed-off-by: Richard Henderson <r...@twiddle.net> > --- > target/arm/translate.h | 8 +--- > target/arm/translate-a64.c | 107 ++++++++----------------------------------- > target/arm/translate.c | 110 > ++++++++++----------------------------------- > 3 files changed, 42 insertions(+), 183 deletions(-)
(snip) > diff --git a/target/arm/translate.c b/target/arm/translate.c > index 4ea5f70..4b1230b 100644 > --- a/target/arm/translate.c > +++ b/target/arm/translate.c > @@ -11897,7 +11897,9 @@ static void arm_tr_tb_start(DisasContextBase *dcbase, > CPUState *cpu) > TCGv_i32 tmp = tcg_temp_new_i32(); > tcg_gen_movi_i32(tmp, 0); > store_cpu_field(tmp, condexec_bits); > + tcg_temp_free_i32(tmp); This seems unrelated to the patch. Perhaps a better place to add this fix would be patch 20, with a mention to it in the commit log. Or just keep it here, but mention it in the commit log. Otherwise, Reviewed-by: Emilio G. Cota <c...@braap.org>