Patchwork RFA: Fix bootstrap/46358

login
register
mail settings
Submitter Ian Taylor
Date Nov. 8, 2010, 6:11 p.m.
Message ID <mcr7hgn4rdr.fsf@google.com>
Download mbox | patch
Permalink /patch/70439/
State New
Headers show

Comments

Ian Taylor - Nov. 8, 2010, 6:11 p.m.
amylaar@spamcop.net writes:

> Quoting Richard Henderson <rth@redhat.com>:
>
>> I don't see that in my tree.
>
> I mean the current version in svn already casts it to HOST_WIDE_INT.

I think Richard meant something more like this (this patch is against
current trunk).

Ian

Patch

Index: config/i386/i386.c
===================================================================
--- config/i386/i386.c	(revision 166445)
+++ config/i386/i386.c	(working copy)
@@ -11063,7 +11063,7 @@  ix86_expand_split_stack_prologue (void)
 {
   struct ix86_frame frame;
   HOST_WIDE_INT allocate;
-  int args_size;
+  HOST_WIDE_INT args_size;
   rtx label, limit, current, jump_insn, allocate_rtx, call_insn, call_fusage;
   rtx scratch_reg = NULL_RTX;
   rtx varargs_label = NULL_RTX;
@@ -11172,8 +11172,7 @@  ix86_expand_split_stack_prologue (void)
 	     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 (((unsigned HOST_WIDE_INT) args_size & 0xffffffff)
-		      == (unsigned HOST_WIDE_INT) args_size);
+	  gcc_assert ((args_size & (HOST_WIDE_INT) 0xffffffff) == args_size);
 
 	  if (split_stack_fn_large == NULL_RTX)
 	    split_stack_fn_large =
@@ -11202,7 +11201,7 @@  ix86_expand_split_stack_prologue (void)
 
 	  fn = reg11;
 
-	  argval = (((HOST_WIDE_INT) args_size << 16) << 16) + allocate;
+	  argval = ((args_size << 16) << 16) + allocate;
 	  emit_move_insn (reg10, GEN_INT (argval));
 	}
       else