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  <rguent...@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  <rguent...@suse.de>

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

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));
 }
 

Reply via email to