diff options
author | Joseph Myers <jsm@polyomino.org.uk> | 2004-05-31 23:06:33 +0100 |
---|---|---|
committer | Joseph Myers <jsm28@gcc.gnu.org> | 2004-05-31 23:06:33 +0100 |
commit | ec3e68358fa934bd54bf6c93af37360b30dfff70 (patch) | |
tree | 10179230ae3065bb71c868f953b6d2060666594b | |
parent | a4a90b847aac34b2cbede88ffeef45343214d066 (diff) | |
download | gcc-ec3e68358fa934bd54bf6c93af37360b30dfff70.zip gcc-ec3e68358fa934bd54bf6c93af37360b30dfff70.tar.gz gcc-ec3e68358fa934bd54bf6c93af37360b30dfff70.tar.bz2 |
re PR c/15749 (--pedantic-errors behaves differently from --pedantic with C-compiler on Linux)
PR c/15749
* c-decl.c (grokdeclarator, finish_struct): Don't pedwarn for
misuses of structures with flexible array members if
in_system_header.
testsuite:
* gcc.dg/pr15749-1.c, gcc.dg/pr15749-1.h: New test.
From-SVN: r82506
-rw-r--r-- | gcc/ChangeLog | 7 | ||||
-rw-r--r-- | gcc/c-decl.c | 4 | ||||
-rw-r--r-- | gcc/testsuite/ChangeLog | 5 | ||||
-rw-r--r-- | gcc/testsuite/gcc.dg/pr15749-1.c | 8 | ||||
-rw-r--r-- | gcc/testsuite/gcc.dg/pr15749-1.h | 28 |
5 files changed, 50 insertions, 2 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 318d3bd..5d10d0f 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2004-05-31 Joseph S. Myers <jsm@polyomino.org.uk> + + PR c/15749 + * c-decl.c (grokdeclarator, finish_struct): Don't pedwarn for + misuses of structures with flexible array members if + in_system_header. + 2004-05-31 Eric Botcazou <ebotcazou@libertysurf.fr> PR target/15693 diff --git a/gcc/c-decl.c b/gcc/c-decl.c index 5cf4e0e..3a13664 100644 --- a/gcc/c-decl.c +++ b/gcc/c-decl.c @@ -3928,7 +3928,7 @@ grokdeclarator (tree declarator, tree declspecs, type = error_mark_node; } - if (pedantic && flexible_array_type_p (type)) + if (pedantic && !in_system_header && flexible_array_type_p (type)) pedwarn ("invalid use of structure with flexible array member"); if (size == error_mark_node) @@ -5204,7 +5204,7 @@ finish_struct (tree t, tree fieldlist, tree attributes) } } - if (pedantic && TREE_CODE (t) == RECORD_TYPE + if (pedantic && !in_system_header && TREE_CODE (t) == RECORD_TYPE && flexible_array_type_p (TREE_TYPE (x))) pedwarn ("%Jinvalid use of structure with flexible array member", x); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 19d8b06..a337797 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2004-05-31 Joseph S. Myers <jsm@polyomino.org.uk> + + PR c/15749 + * gcc.dg/pr15749-1.c, gcc.dg/pr15749-1.h: New test. + 2004-05-31 Eric Botcazou <ebotcazou@libertysurf.fr> * gcc.dg/sparc-trap-1.c: New test. diff --git a/gcc/testsuite/gcc.dg/pr15749-1.c b/gcc/testsuite/gcc.dg/pr15749-1.c new file mode 100644 index 0000000..e78816f --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr15749-1.c @@ -0,0 +1,8 @@ +/* Flexible array misuses (that are accepted without -pedantic) should + be OK in system headers even with -pedantic-errors. PR 15749 + from Tuomo dot Tikkanen at nokia dot com. */ +/* Origin: Joseph Myers <jsm@polyomino.org.uk> */ +/* { dg-do compile } */ +/* { dg-options "-std=iso9899:1999 -pedantic-errors" } */ + +#include "pr15749-1.h" diff --git a/gcc/testsuite/gcc.dg/pr15749-1.h b/gcc/testsuite/gcc.dg/pr15749-1.h new file mode 100644 index 0000000..db4a259 --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr15749-1.h @@ -0,0 +1,28 @@ +/* Similar to c99-flex-array-3.c, but a system header so should not + have diagnostics even with -pedantic-errors. */ + +#pragma GCC system_header + +struct flex { int a; int b[]; }; +union rf1 { struct flex a; int b; }; +union rf2 { int a; struct flex b; }; +union rf3 { int a; union rf1 b; }; +union rf4 { union rf2 a; int b; }; + +struct t0 { struct flex a; }; +struct t1 { union rf1 a; }; +struct t2 { union rf2 a; }; +struct t3 { union rf3 a; }; +struct t4 { union rf4 a; }; + +void f0 (struct flex[]); +void f1 (union rf1[]); +void f2 (union rf2[]); +void f3 (union rf3[]); +void f4 (union rf4[]); + +struct flex a0[1]; +union rf1 a1[1]; +union rf2 a2[1]; +union rf3 a3[1]; +union rf4 a4[1]; |