diff --git a/gcc/ChangeLog b/gcc/ChangeLog index c4f8b626fe21ff959f7c6098de02276e29d1e05e..8e2759752a0183d4e7f729cc44160659ae917b31 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,5 +1,9 @@ 2006-01-04 Jakub Jelinek <jakub@redhat.com> + PR c/25559 + * c-common.c (handle_vector_size_attribute): Reject zero vector size + as well as sizes not multiple of component size. + PR debug/25562 * function.c (instantiate_expr): New function. (instantiate_decls_1, instantiate_decls): If DECL_HAS_VALUE_EXPR_P, diff --git a/gcc/c-common.c b/gcc/c-common.c index b252345945840abad3100a312aa70b41369c7f9e..76e90962f9bcc0dfa81fb11ade1352991442653b 100644 --- a/gcc/c-common.c +++ b/gcc/c-common.c @@ -5199,6 +5199,18 @@ handle_vector_size_attribute (tree *node, tree name, tree args, return NULL_TREE; } + if (vecsize % tree_low_cst (TYPE_SIZE_UNIT (type), 1)) + { + error ("vector size not an integral multiple of component size"); + return NULL; + } + + if (vecsize == 0) + { + error ("zero vector size"); + return NULL; + } + /* Calculate how many units fit in the vector. */ nunits = vecsize / tree_low_cst (TYPE_SIZE_UNIT (type), 1); if (nunits & (nunits - 1)) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index e02c3c7a5104debeed5be6b3804c4adf5fe3e394..23350e1ca5549971263c30e1af7e87731eedec54 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2006-01-04 Jakub Jelinek <jakub@redhat.com> + + PR c/25559 + * gcc.dg/pr25559.c: New test. + 2006-01-03 Mark Mitchell <mark@codesourcery.com> PR c++/25492 diff --git a/gcc/testsuite/gcc.dg/pr25559.c b/gcc/testsuite/gcc.dg/pr25559.c new file mode 100644 index 0000000000000000000000000000000000000000..7879a1558b64b7c2b707a5c25b5fbea2ebd0f041 --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr25559.c @@ -0,0 +1,10 @@ +/* PR c/25559 */ +/* { dg-do compile } */ + +#define vs(n) __attribute__((vector_size (n))) +int vs (-1) a; /* { dg-warning "attribute ignored" } */ +int vs (0) b; /* { dg-error "zero vector size" } */ +int vs (1) c; /* { dg-error "multiple of component size" } */ +int vs (sizeof (int) / 2) d; /* { dg-error "multiple of component size" } */ +int vs (sizeof (int)) e; +int vs (sizeof (int) * 2) f;