Re: [Qemu-devel] [PATCH v2 08/12] target-s390x: Add missing tcg_temp_free_i64() in disas_a5(), opc == 0x8

[Alexander Graf]

On 27.05.2011, at 19:03, Stefan Weil wrote:

> load_reg() needs a matching tcg_temp_free_i64().
> 
> Signed-off-by: Stefan Weil <address@hidden>
> ---
> target-s390x/translate.c |    1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/target-s390x/translate.c b/target-s390x/translate.c
> index 81b8c5b..692de6e 100644
> --- a/target-s390x/translate.c
> +++ b/target-s390x/translate.c
> @@ -2365,6 +2365,7 @@ static void disas_a5(DisasContext *s, int op, int r1, 
> int i2)
>         tcg_gen_shri_i64(tmp2, tmp, 48);
>         tcg_gen_trunc_i64_i32(tmp32, tmp2);
>         set_cc_nz_u32(s, tmp32);
> +        tcg_temp_free_i64(tmp);

tmp gets freed at the end of the function, so this one is bad.


Alex