[PATCH-for-8.0 0/2] target/tcg: Use TCGv_i64 with tcg_temp_new_i64()

[PATCH-for-8.0 0/2] target/tcg: Use TCGv_i64 with tcg_temp_new_i64()

[Philippe Mathieu-Daudé]

Although on 64-bit targets TCGv is defined as TCGv_i64,
clarify using the correct type for tcg_temp_new_i64().

Philippe Mathieu-Daudé (2):
  target/s390x: Replace TCGv by TCGv_i64 in op_mov2e()
  target/sparc64: Replace TCGv by TCGv_i64 in gen_op_multiply()

 target/s390x/tcg/translate.c | 2 +-
 target/sparc/translate.c     | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

-- 
2.38.1

[PATCH-for-8.0 1/2] target/s390x: Replace TCGv by TCGv_i64 in op_mov2e()

[Philippe Mathieu-Daudé]

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é <philmd@linaro.org>
---
 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;
-- 
2.38.1

Re: [PATCH-for-8.0 1/2] target/s390x: Replace TCGv by TCGv_i64 in op_mov2e()

[Ilya Leoshkevich]

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é <philmd@linaro.org>
> ---
>  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 <iii@linux.ibm.com>

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();

Re: [PATCH-for-8.0 1/2] target/s390x: Replace TCGv by TCGv_i64 in op_mov2e()

[Philippe Mathieu-Daudé]

On 1/12/22 13:36, Ilya Leoshkevich wrote:
> 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é <philmd@linaro.org>
>> ---
>>   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 <iii@linux.ibm.com>
> 
> 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();

Oh I missed it, thanks!

[PATCH-for-8.0 2/2] target/sparc64: Replace TCGv by TCGv_i64 in gen_op_multiply()

[Philippe Mathieu-Daudé]

Although TCGv is defined as TCGv_i64 on sparc64,
make it clear tcg_temp_new_i64() returns a TCGv_i64.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 target/sparc/translate.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/target/sparc/translate.c b/target/sparc/translate.c
index 34858eb95f..5a08641995 100644
--- a/target/sparc/translate.c
+++ b/target/sparc/translate.c
@@ -647,8 +647,8 @@ static inline void gen_op_multiply(TCGv dst, TCGv src1, TCGv src2, int sign_ext)
         tcg_gen_mulu2_tl(dst, cpu_y, src1, src2);
     }
 #else
-    TCGv t0 = tcg_temp_new_i64();
-    TCGv t1 = tcg_temp_new_i64();
+    TCGv_i64 t0 = tcg_temp_new_i64();
+    TCGv_i64 t1 = tcg_temp_new_i64();
 
     if (sign_ext) {
         tcg_gen_ext32s_i64(t0, src1);
-- 
2.38.1

Re: [PATCH-for-8.0 0/2] target/tcg: Use TCGv_i64 with tcg_temp_new_i64()

[Richard Henderson]

On 11/30/22 08:34, Philippe Mathieu-Daudé wrote:
> Although on 64-bit targets TCGv is defined as TCGv_i64,
> clarify using the correct type for tcg_temp_new_i64().
> 
> Philippe Mathieu-Daudé (2):
>    target/s390x: Replace TCGv by TCGv_i64 in op_mov2e()
>    target/sparc64: Replace TCGv by TCGv_i64 in gen_op_multiply()
> 
>   target/s390x/tcg/translate.c | 2 +-
>   target/sparc/translate.c     | 4 ++--
>   2 files changed, 3 insertions(+), 3 deletions(-)
> 

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~