c-parse.in (stmts_and_decls): Make label at end of compound statement a hard error.

	* c-parse.in (stmts_and_decls): Make label at end of compound
	statement a hard error.

testsuite:
	* gcc.dg/label-compound-stmt-1.c: New test.
	* gcc.c-torture/compile/950922-1.c,
	gcc.c-torture/compile/20000211-3.c,
	gcc.c-torture/compile/20000518-1.c,
	gcc.c-torture/compile/20021108-1.c: Avoid labels at and of
	compound statements.

From-SVN: r75891

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 29bcb0631d393c089bfa0640659a45286e8b4102..e5d53961ba256ac60bbaaa8c47137d565ad7e8bc 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2004-01-14  Joseph S. Myers  <jsm@polyomino.org.uk>
+
+       * c-parse.in (stmts_and_decls): Make label at end of compound
+       statement a hard error.
+
 2004-01-14  Jan Hubicka  <jh@suse.cz>
 
        * cgraph.c (create_edge): Use local.redefined_extern_inline.

diff --git a/gcc/c-parse.in b/gcc/c-parse.in
index 954c37067ffa99578f962d1d0e24842b2e7c0358..4cdbfed8b0c6a68b14637aff34a113a793b3d89e 100644 (file)
--- a/gcc/c-parse.in
+++ b/gcc/c-parse.in
@@ -1,6 +1,6 @@
 /* YACC parser for C syntax and for Objective C.  -*-c-*-
-   Copyright (C) 1987, 1988, 1989, 1992, 1993, 1994, 1995, 1996,
-   1997, 1998, 1999, 2000, 2001, 2002, 2003 Free Software Foundation, Inc.
+   Copyright (C) 1987, 1988, 1989, 1992, 1993, 1994, 1995, 1996, 1997,
+   1998, 1999, 2000, 2001, 2002, 2003, 2004 Free Software Foundation, Inc.
 
 This file is part of GCC.
 
@@ -2001,7 +2001,7 @@ stmts_and_decls:
        | lineno_stmt_decl_or_labels_ending_decl
        | lineno_stmt_decl_or_labels_ending_label
                {
-                 pedwarn ("deprecated use of label at end of compound statement");
+                 error ("label at end of compound statement");
                }
        | lineno_stmt_decl_or_labels_ending_error
        ;

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 05a6b6027c21f7d0cf60b4302828f268096995f1..2ac3cc83f98db5d65bdff834c5f2c53be458ac3a 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,12 @@
+2004-01-14  Joseph S. Myers  <jsm@polyomino.org.uk>
+
+       * gcc.dg/label-compound-stmt-1.c: New test.
+       * gcc.c-torture/compile/950922-1.c,
+       gcc.c-torture/compile/20000211-3.c,
+       gcc.c-torture/compile/20000518-1.c,
+       gcc.c-torture/compile/20021108-1.c: Avoid labels at and of
+       compound statements.
+
 2004-01-14  Hartmut Penner  <hpenner@de.ibm.com>
 
        * gcc.dg/ppc64-abi-1.c: New test.

diff --git a/gcc/testsuite/gcc.c-torture/compile/20000211-3.c b/gcc/testsuite/gcc.c-torture/compile/20000211-3.c
index 7e43c8ad46708fdf3abe7192c842a157ef355c5a..7a02b685c4ab6813b605aefa82e2fb5077abfc9a 100644 (file)
--- a/gcc/testsuite/gcc.c-torture/compile/20000211-3.c
+++ b/gcc/testsuite/gcc.c-torture/compile/20000211-3.c
@@ -3,7 +3,7 @@ void f_clos(int x)
 {        
         switch(x) {
                 default:
-                mumble:
+                mumble:;
         }
 }
 

diff --git a/gcc/testsuite/gcc.c-torture/compile/20000518-1.c b/gcc/testsuite/gcc.c-torture/compile/20000518-1.c
index fda44b657cce7570e3510fc52d30b27840c409f5..d2087796a8473850ae9000a9a70f0d280ed5a905 100644 (file)
--- a/gcc/testsuite/gcc.c-torture/compile/20000518-1.c
+++ b/gcc/testsuite/gcc.c-torture/compile/20000518-1.c
@@ -6,7 +6,7 @@ extern __inline__ void test()
 
        callit1(&&l1);
 
-l1:
+l1:;
 
 }
 

diff --git a/gcc/testsuite/gcc.c-torture/compile/20021108-1.c b/gcc/testsuite/gcc.c-torture/compile/20021108-1.c
index c421639035a8c2435c220b21a2a9ddb842869c19..01b6e4ed7aa84b2529aa058addbf1d237045e50e 100644 (file)
--- a/gcc/testsuite/gcc.c-torture/compile/20021108-1.c
+++ b/gcc/testsuite/gcc.c-torture/compile/20021108-1.c
@@ -3,5 +3,5 @@ main()
 {
 l1:
   return &&l1-&&l2;
-l2:
+l2:;
 }

diff --git a/gcc/testsuite/gcc.c-torture/compile/950922-1.c b/gcc/testsuite/gcc.c-torture/compile/950922-1.c
index 73574f33e68e0e7851dac0a28b03f4e8dddb7507..73c52f7740a17c6bd6de569f5f47f542ca520090 100644 (file)
--- a/gcc/testsuite/gcc.c-torture/compile/950922-1.c
+++ b/gcc/testsuite/gcc.c-torture/compile/950922-1.c
@@ -40,7 +40,7 @@ f (int ch, char *fp, char *ap)
     nosign:
       if (_uquad != 0 || prec != 0);
       break;
-    default:
+    default:;
     }
     if ((f & 0x100) == 0) {
     } else {
@@ -64,5 +64,5 @@ f (int ch, char *fp, char *ap)
     }
   }
 
- error:
+ error:;
 }

diff --git a/gcc/testsuite/gcc.dg/label-compound-stmt-1.c b/gcc/testsuite/gcc.dg/label-compound-stmt-1.c
new file mode 100644 (file)
index 0000000..7141163
--- /dev/null
+++ b/gcc/testsuite/gcc.dg/label-compound-stmt-1.c
@@ -0,0 +1,7 @@
+/* Test that labels at ends of compound statements are hard errors.  */
+/* Origin: Joseph Myers <jsm@polyomino.org.uk> */
+/* { dg-do compile } */
+/* { dg-options "" } */
+
+void f(void) { g: } /* { dg-bogus "warning" "warning in place of error" } */
+/* { dg-error "label|parse|syntax" "label at end of compound statement" { target *-*-* } 6 } */