diff mbox series

[PATCH-for-8.0,1/3] tcg/s390x: Fix coding style

Message ID 20221130132654.76369-2-philmd@linaro.org (mailing list archive)
State New, archived
Headers show
Series tcg: Replace tcg_target_[u]long -> [u]intptr_t | expand

Commit Message

Philippe Mathieu-Daudé Nov. 30, 2022, 1:26 p.m. UTC 
We are going to modify this code, so fix its style first to avoid:

  ERROR: spaces required around that '*' (ctx:VxV)
  #281: FILE: tcg/s390x/tcg-target.c.inc:1224:
  +        uintptr_t mask = ~(0xffffull << i*16);
                                            ^

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 tcg/s390x/tcg-target.c.inc | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

Comments

Wilfred Mallawa Dec. 1, 2022, 3:22 a.m. UTC | #1 
On Wed, 2022-11-30 at 14:26 +0100, Philippe Mathieu-Daudé wrote:
> We are going to modify this code, so fix its style first to avoid:
> 
>   ERROR: spaces required around that '*' (ctx:VxV)
>   #281: FILE: tcg/s390x/tcg-target.c.inc:1224:
>   +        uintptr_t mask = ~(0xffffull << i*16);
>                                             ^
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>  tcg/s390x/tcg-target.c.inc | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
Reviewed-by: Wilfred Mallawa <wilfred.mallawa@wdc.com>
> 
> diff --git a/tcg/s390x/tcg-target.c.inc b/tcg/s390x/tcg-target.c.inc
> index 33becd7694..f1d3907cd8 100644
> --- a/tcg/s390x/tcg-target.c.inc
> +++ b/tcg/s390x/tcg-target.c.inc
> @@ -802,9 +802,9 @@ static bool maybe_out_small_movi(TCGContext *s,
> TCGType type,
>      }
>  
>      for (i = 0; i < 4; i++) {
> -        tcg_target_long mask = 0xffffull << i*16;
> +        tcg_target_long mask = 0xffffull << i * 16;
>          if ((uval & mask) == uval) {
> -            tcg_out_insn_RI(s, lli_insns[i], ret, uval >> i*16);
> +            tcg_out_insn_RI(s, lli_insns[i], ret, uval >> i * 16);
>              return true;
>          }
>      }
> @@ -1221,9 +1221,9 @@ static void tgen_andi(TCGContext *s, TCGType
> type, TCGReg dest, uint64_t val)
>  
>      /* Try all 32-bit insns that can perform it in one go.  */
>      for (i = 0; i < 4; i++) {
> -        tcg_target_ulong mask = ~(0xffffull << i*16);
> +        tcg_target_ulong mask = ~(0xffffull << i * 16);
>          if (((val | ~valid) & mask) == mask) {
> -            tcg_out_insn_RI(s, ni_insns[i], dest, val >> i*16);
> +            tcg_out_insn_RI(s, ni_insns[i], dest, val >> i * 16);
>              return;
>          }
>      }
> @@ -1231,9 +1231,9 @@ static void tgen_andi(TCGContext *s, TCGType
> type, TCGReg dest, uint64_t val)
>      /* Try all 48-bit insns that can perform it in one go.  */
>      if (HAVE_FACILITY(EXT_IMM)) {
>          for (i = 0; i < 2; i++) {
> -            tcg_target_ulong mask = ~(0xffffffffull << i*32);
> +            tcg_target_ulong mask = ~(0xffffffffull << i * 32);
>              if (((val | ~valid) & mask) == mask) {
> -                tcg_out_insn_RIL(s, nif_insns[i], dest, val >>
> i*32);
> +                tcg_out_insn_RIL(s, nif_insns[i], dest, val >> i *
> 32);
>                  return;
>              }
>          }
> @@ -1279,9 +1279,9 @@ static void tgen_ori(TCGContext *s, TCGType
> type, TCGReg dest, uint64_t val)
>  
>      /* Try all 32-bit insns that can perform it in one go.  */
>      for (i = 0; i < 4; i++) {
> -        tcg_target_ulong mask = (0xffffull << i*16);
> +        tcg_target_ulong mask = (0xffffull << i * 16);
>          if ((val & mask) != 0 && (val & ~mask) == 0) {
> -            tcg_out_insn_RI(s, oi_insns[i], dest, val >> i*16);
> +            tcg_out_insn_RI(s, oi_insns[i], dest, val >> i * 16);
>              return;
>          }
>      }
> @@ -1289,9 +1289,9 @@ static void tgen_ori(TCGContext *s, TCGType
> type, TCGReg dest, uint64_t val)
>      /* Try all 48-bit insns that can perform it in one go.  */
>      if (HAVE_FACILITY(EXT_IMM)) {
>          for (i = 0; i < 2; i++) {
> -            tcg_target_ulong mask = (0xffffffffull << i*32);
> +            tcg_target_ulong mask = (0xffffffffull << i * 32);
>              if ((val & mask) != 0 && (val & ~mask) == 0) {
> -                tcg_out_insn_RIL(s, oif_insns[i], dest, val >>
> i*32);
> +                tcg_out_insn_RIL(s, oif_insns[i], dest, val >> i *
> 32);
>                  return;
>              }
>          }
Richard Henderson Dec. 1, 2022, 6:36 a.m. UTC | #2 
On 11/30/22 05:26, Philippe Mathieu-Daudé wrote:
> We are going to modify this code, so fix its style first to avoid:
> 
>    ERROR: spaces required around that '*' (ctx:VxV)
>    #281: FILE: tcg/s390x/tcg-target.c.inc:1224:
>    +        uintptr_t mask = ~(0xffffull << i*16);
>                                              ^
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   tcg/s390x/tcg-target.c.inc | 20 ++++++++++----------
>   1 file changed, 10 insertions(+), 10 deletions(-)


Thanks, queued to tcg-next.


r~
diff mbox series

Patch

diff --git a/tcg/s390x/tcg-target.c.inc b/tcg/s390x/tcg-target.c.inc
index 33becd7694..f1d3907cd8 100644
--- a/tcg/s390x/tcg-target.c.inc
+++ b/tcg/s390x/tcg-target.c.inc
@@ -802,9 +802,9 @@  static bool maybe_out_small_movi(TCGContext *s, TCGType type,
     }
 
     for (i = 0; i < 4; i++) {
-        tcg_target_long mask = 0xffffull << i*16;
+        tcg_target_long mask = 0xffffull << i * 16;
         if ((uval & mask) == uval) {
-            tcg_out_insn_RI(s, lli_insns[i], ret, uval >> i*16);
+            tcg_out_insn_RI(s, lli_insns[i], ret, uval >> i * 16);
             return true;
         }
     }
@@ -1221,9 +1221,9 @@  static void tgen_andi(TCGContext *s, TCGType type, TCGReg dest, uint64_t val)
 
     /* Try all 32-bit insns that can perform it in one go.  */
     for (i = 0; i < 4; i++) {
-        tcg_target_ulong mask = ~(0xffffull << i*16);
+        tcg_target_ulong mask = ~(0xffffull << i * 16);
         if (((val | ~valid) & mask) == mask) {
-            tcg_out_insn_RI(s, ni_insns[i], dest, val >> i*16);
+            tcg_out_insn_RI(s, ni_insns[i], dest, val >> i * 16);
             return;
         }
     }
@@ -1231,9 +1231,9 @@  static void tgen_andi(TCGContext *s, TCGType type, TCGReg dest, uint64_t val)
     /* Try all 48-bit insns that can perform it in one go.  */
     if (HAVE_FACILITY(EXT_IMM)) {
         for (i = 0; i < 2; i++) {
-            tcg_target_ulong mask = ~(0xffffffffull << i*32);
+            tcg_target_ulong mask = ~(0xffffffffull << i * 32);
             if (((val | ~valid) & mask) == mask) {
-                tcg_out_insn_RIL(s, nif_insns[i], dest, val >> i*32);
+                tcg_out_insn_RIL(s, nif_insns[i], dest, val >> i * 32);
                 return;
             }
         }
@@ -1279,9 +1279,9 @@  static void tgen_ori(TCGContext *s, TCGType type, TCGReg dest, uint64_t val)
 
     /* Try all 32-bit insns that can perform it in one go.  */
     for (i = 0; i < 4; i++) {
-        tcg_target_ulong mask = (0xffffull << i*16);
+        tcg_target_ulong mask = (0xffffull << i * 16);
         if ((val & mask) != 0 && (val & ~mask) == 0) {
-            tcg_out_insn_RI(s, oi_insns[i], dest, val >> i*16);
+            tcg_out_insn_RI(s, oi_insns[i], dest, val >> i * 16);
             return;
         }
     }
@@ -1289,9 +1289,9 @@  static void tgen_ori(TCGContext *s, TCGType type, TCGReg dest, uint64_t val)
     /* Try all 48-bit insns that can perform it in one go.  */
     if (HAVE_FACILITY(EXT_IMM)) {
         for (i = 0; i < 2; i++) {
-            tcg_target_ulong mask = (0xffffffffull << i*32);
+            tcg_target_ulong mask = (0xffffffffull << i * 32);
             if ((val & mask) != 0 && (val & ~mask) == 0) {
-                tcg_out_insn_RIL(s, oif_insns[i], dest, val >> i*32);
+                tcg_out_insn_RIL(s, oif_insns[i], dest, val >> i * 32);
                 return;
             }
         }