re PR c++/24138 (ICE with the code in PR 20407)

PR C++/24138
        * tree.c (integer_all_onesp): Always return true if all bits on.

        * cp/decl.c (reshape_init_array_1): Handle max_index of -1.

        * testsuite/g++.dg/init/array0.C: New.

From-SVN: r108126
This commit is contained in:
Aldy Hernandez 2005-12-06 19:45:00 +00:00 committed by Aldy Hernandez
parent 3af22b23b3
commit d7d9383746
5 changed files with 32 additions and 2 deletions

View File

@ -1,3 +1,10 @@
2005-12-06 Aldy Hernandez <aldyh@redhat.com>
PR C++/24138
* tree.c (integer_all_onesp): Always return true if all bits on.
* testsuite/g++.dg/init/array0.C: New.
2005-12-06 Adrian Straetling <straetling@de.ibm.com>
* doc/md.texi: Adapt to implementation.

View File

@ -1,3 +1,8 @@
2005-12-06 Aldy Hernandez <aldyh@redhat.com>
PR C++/24138
* decl.c (reshape_init_array_1): Handle max_index of -1.
2005-12-06 Roger Sayle <roger@eyesopen.com>
* typeck.c (build_binary_op): Issue warning if either operand of a

View File

@ -4209,6 +4209,10 @@ reshape_init_array_1 (tree elt_type, tree max_index, reshape_iter *d)
if (sized_array_p)
{
/* Minus 1 is used for zero sized arrays. */
if (integer_all_onesp (max_index))
return new_init;
if (host_integerp (max_index, 1))
max_index_cst = tree_low_cst (max_index, 1);
/* sizetype is sign extended, not zero extended. */

View File

@ -0,0 +1,12 @@
// { dg-do compile }
// { dg-options "" }
// PR C++/24138
void foo()
{
typedef struct {
unsigned char dir;
int data[0];
} yanito;
static const yanito horse = { 1, { 2, 3 } }; // { dg-error "too many" }
}

View File

@ -1208,9 +1208,11 @@ integer_all_onesp (tree expr)
return 0;
uns = TYPE_UNSIGNED (TREE_TYPE (expr));
if (TREE_INT_CST_LOW (expr) == ~(unsigned HOST_WIDE_INT) 0
&& TREE_INT_CST_HIGH (expr) == -1)
return 1;
if (!uns)
return (TREE_INT_CST_LOW (expr) == ~(unsigned HOST_WIDE_INT) 0
&& TREE_INT_CST_HIGH (expr) == -1);
return 0;
/* Note that using TYPE_PRECISION here is wrong. We care about the
actual bits, not the (arbitrary) range of the type. */