Patchwork Fix bootstrap issue on 32bit HWI hosts

login
register
mail settings
Submitter Richard Guenther
Date July 1, 2010, 10:21 a.m.
Message ID <alpine.LNX.2.00.1007011220160.1429@zhemvz.fhfr.qr>
Download mbox | patch
Permalink /patch/57499/
State New
Headers show

Comments

Richard Guenther - July 1, 2010, 10:21 a.m.
This should address the issue seen by Geoffs tester.  I am in
the process of 1) reproducing the issue and 2) verifying the
fix below fixes the issue.  The patch below has already been
bootstrapped on x86_64-unknown-linux-gnu.

Richard.

2010-07-01  Richard Guenther  <rguenther@suse.de>

	* emit-rtl.c (set_mem_attributes_minus_bitpos): Use unsigned
	types for offsets.
Richard Guenther - July 1, 2010, 10:51 a.m.
On Thu, 1 Jul 2010, Richard Guenther wrote:

> 
> This should address the issue seen by Geoffs tester.  I am in
> the process of 1) reproducing the issue and 2) verifying the
> fix below fixes the issue.  The patch below has already been
> bootstrapped on x86_64-unknown-linux-gnu.

Done 1) and 2) with verifying we are going into stage3 without
any errors.

Applied as rev. 161659.

Richard.

Patch

Index: gcc/emit-rtl.c
===================================================================
--- gcc/emit-rtl.c	(revision 161657)
+++ gcc/emit-rtl.c	(working copy)
@@ -1616,10 +1616,10 @@  set_mem_attributes_minus_bitpos (rtx ref
     align = MAX (align, TYPE_ALIGN (type));
   else if (TREE_CODE (t) == MEM_REF)
     {
-      HOST_WIDE_INT aoff = BITS_PER_UNIT;
+      unsigned HOST_WIDE_INT aoff = BITS_PER_UNIT;
       if (host_integerp (TREE_OPERAND (t, 1), 1))
 	{
-	  HOST_WIDE_INT ioff = TREE_INT_CST_LOW (TREE_OPERAND (t, 1));
+	  unsigned HOST_WIDE_INT ioff = TREE_INT_CST_LOW (TREE_OPERAND (t, 1));
 	  aoff = (ioff & -ioff) * BITS_PER_UNIT;
 	}
       if (TREE_CODE (TREE_OPERAND (t, 0)) == ADDR_EXPR