On Wed, 2022-11-30 at 17:34 +0100, Philippe Mathieu-Daudé wrote: > Although TCGv is defined as TCGv_i64 on s390x, > make it clear tcg_temp_new_i64() returns a TCGv_i64. > > Signed-off-by: Philippe Mathieu-Daudé <[email protected]> > --- > target/s390x/tcg/translate.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/target/s390x/tcg/translate.c > b/target/s390x/tcg/translate.c > index 1e599ac259..a77039b863 100644 > --- a/target/s390x/tcg/translate.c > +++ b/target/s390x/tcg/translate.c > @@ -3335,7 +3335,7 @@ static DisasJumpType op_mov2(DisasContext *s, > DisasOps *o) > static DisasJumpType op_mov2e(DisasContext *s, DisasOps *o) > { > int b2 = get_field(s, b2); > - TCGv ar1 = tcg_temp_new_i64(); > + TCGv_i64 ar1 = tcg_temp_new_i64(); > > o->out = o->in2; > o->g_out = o->g_in2;
Reviewed-by: Ilya Leoshkevich <[email protected]> It looks as if besides sparc and s390x there is one occurrence of this in alpha? $ git grep -w TCGv | grep -w tcg_temp_new_i64 target/alpha/translate.c: TCGv tmp = tcg_temp_new_i64();
