This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR53970
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.
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));
}