Hi!

I found a couple of spots where we can use the HOST_WIDE_INT_C macro.

Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?

2016-05-13  Jakub Jelinek  <ja...@redhat.com>

        * config/i386/i386.c (ix86_compute_frame_layout, ix86_expand_prologue,
        ix86_expand_split_stack_prologue): Use HOST_WIDE_INT_C macro.
        (ix86_split_to_parts): Likewise.  Fix up formatting.

--- gcc/config/i386/i386.c.jj   2016-05-12 09:46:40.000000000 +0200
+++ gcc/config/i386/i386.c      2016-05-12 10:46:35.862566575 +0200
@@ -11957,7 +11957,7 @@ ix86_compute_frame_layout (struct ix86_f
   to_allocate = offset - frame->sse_reg_save_offset;
 
   if ((!to_allocate && frame->nregs <= 1)
-      || (TARGET_64BIT && to_allocate >= (HOST_WIDE_INT) 0x80000000))
+      || (TARGET_64BIT && to_allocate >= HOST_WIDE_INT_C (0x80000000)))
     frame->save_regs_using_mov = false;
 
   if (ix86_using_red_zone ()
@@ -13379,7 +13379,7 @@ ix86_expand_prologue (void)
        {
          HOST_WIDE_INT size = allocate;
 
-         if (TARGET_64BIT && size >= (HOST_WIDE_INT) 0x80000000)
+         if (TARGET_64BIT && size >= HOST_WIDE_INT_C (0x80000000))
            size = 0x80000000 - STACK_CHECK_PROTECT - 1;
 
          if (TARGET_STACK_PROBE)
@@ -14320,7 +14320,7 @@ ix86_expand_split_stack_prologue (void)
             different function: __morestack_large.  We pass the
             argument size in the upper 32 bits of r10 and pass the
             frame size in the lower 32 bits.  */
-         gcc_assert ((allocate & (HOST_WIDE_INT) 0xffffffff) == allocate);
+         gcc_assert ((allocate & HOST_WIDE_INT_C (0xffffffff)) == allocate);
          gcc_assert ((args_size & 0xffffffff) == args_size);
 
          if (split_stack_fn_large == NULL_RTX)
@@ -24554,20 +24554,17 @@ ix86_split_to_parts (rtx operand, rtx *p
              real_to_target (l, CONST_DOUBLE_REAL_VALUE (operand), mode);
 
              /* real_to_target puts 32-bit pieces in each long.  */
-             parts[0] =
-               gen_int_mode
-                 ((l[0] & (HOST_WIDE_INT) 0xffffffff)
-                  | ((l[1] & (HOST_WIDE_INT) 0xffffffff) << 32),
-                  DImode);
+             parts[0] = gen_int_mode ((l[0] & HOST_WIDE_INT_C (0xffffffff))
+                                      | ((l[1] & HOST_WIDE_INT_C (0xffffffff))
+                                         << 32), DImode);
 
              if (upper_mode == SImode)
                parts[1] = gen_int_mode (l[2], SImode);
              else
-               parts[1] =
-                 gen_int_mode
-                   ((l[2] & (HOST_WIDE_INT) 0xffffffff)
-                    | ((l[3] & (HOST_WIDE_INT) 0xffffffff) << 32),
-                    DImode);
+               parts[1]
+                 = gen_int_mode ((l[2] & HOST_WIDE_INT_C (0xffffffff))
+                                 | ((l[3] & HOST_WIDE_INT_C (0xffffffff))
+                                    << 32), DImode);
            }
          else
            gcc_unreachable ();

        Jakub

Reply via email to