On Fri, May 13, 2016 at 7:11 PM, Jakub Jelinek <ja...@redhat.com> wrote: > 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.
OK. Thanks, Uros. > --- 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