Skip to content

Commit

Permalink
target/arm: Free TCG temps in trans_VMOV_64_sp()
Browse files Browse the repository at this point in the history
The function neon_store_reg32() doesn't free the TCG temp that it
is passed, so the caller must do that. We got this right in most
places but forgot to free the TCG temps in trans_VMOV_64_sp().

Cc: qemu-stable@nongnu.org
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-id: 20190827121931.26836-1-peter.maydell@linaro.org
  • Loading branch information
pm215 committed Sep 3, 2019
1 parent 1eb21c4 commit 342d275
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions target/arm/translate-vfp.inc.c
Expand Up @@ -880,8 +880,10 @@ static bool trans_VMOV_64_sp(DisasContext *s, arg_VMOV_64_sp *a)
/* gpreg to fpreg */
tmp = load_reg(s, a->rt);
neon_store_reg32(tmp, a->vm);
tcg_temp_free_i32(tmp);
tmp = load_reg(s, a->rt2);
neon_store_reg32(tmp, a->vm + 1);
tcg_temp_free_i32(tmp);
}

return true;
Expand Down

0 comments on commit 342d275

Please sign in to comment.