Patchwork Fix PR53970

login
register
mail settings
Submitter Richard Guenther
Date July 19, 2012, 8:47 a.m.
Message ID <alpine.LNX.2.00.1207191045550.17088@jbgna.fhfr.qr>
Download mbox | patch
Permalink /patch/171909/
State New
Headers show

Comments

Richard Guenther - July 19, 2012, 8:47 a.m.
On Wed, 18 Jul 2012, John David Anglin wrote:

> On Wed, 18 Jul 2012, Richard Guenther wrote:
> 
> > Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
> > 
> > Richard.
> > 
> > 2012-07-18  Richard Guenther  <rguenther@suse.de>
> > 
> > 	PR tree-optimization/53970
> > 	* tree.h (contains_packed_reference): Remove.
> > 	* expr.c (contains_packed_reference): Likewise.
> > 	* tree-vect-data-refs.c (not_size_aligned): New function.
> 
> ../../gcc/gcc/tree-vect-data-refs.c: In function ???not_size_aligned???:
> ../../gcc/gcc/tree-vect-data-refs.c:1144:4: warning: comparison between signed a
> 
> This causes a bootstrap failure on hppa.

*sigh*, always this disturbing sign-mismatch of tree_low_cst vs.
TREE_INT_CST_LOW.

Committed as obvious.

Richard.

2012-07-19  Richard Guenther  <rguenther@suse.de>

	PR tree-optimization/53970
	* tree-vect-data-refs.c (not_size_aligned): Avoid sign-compare
	warning.

Patch

Index: gcc/tree-vect-data-refs.c
===================================================================
--- gcc/tree-vect-data-refs.c	(revision 189646)
+++ gcc/tree-vect-data-refs.c	(working copy)
@@ -1140,7 +1140,7 @@  not_size_aligned (tree exp)
   if (!host_integerp (TYPE_SIZE (TREE_TYPE (exp)), 1))
     return true;
 
-  return (tree_low_cst (TYPE_SIZE (TREE_TYPE (exp)), 1)
+  return (TREE_INT_CST_LOW (TYPE_SIZE (TREE_TYPE (exp)))
 	  > get_object_alignment (exp));
 }