re PR c++/44991 (default argument with '<' cause compilation error)

	PR c++/44991
	* parser.c (cp_parser_parameter_declaration): Pop parameter decls
	after tentative parsing.

From-SVN: r163629

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index f485d6ff910ab48665843a2f19990d2c44593255..ac879d777d24d5d910b5a42d158e4b1dfcef82c5 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,9 @@
+2010-08-29  Jason Merrill  <jason@redhat.com>
+
+       PR c++/44991
+       * parser.c (cp_parser_parameter_declaration): Pop parameter decls
+       after tentative parsing.
+
 2010-08-22  Joseph Myers  <joseph@codesourcery.com>
 
        * Make-lang.in (g++spec.o): Update dependencies.

diff --git a/gcc/cp/parser.c b/gcc/cp/parser.c
index 6c4fe1de63ba0649e4ba7a7da14e96c4a6b998ac..b71d145adb9867a259f45a6e3252c7dbf99b5d46 100644 (file)
--- a/gcc/cp/parser.c
+++ b/gcc/cp/parser.c
@@ -15564,6 +15564,7 @@ cp_parser_parameter_declaration (cp_parser *parser,
                         the default argument; otherwise the default
                         argument continues.  */
                      bool error = false;
+                     tree t;
 
                      /* Set ITALP so cp_parser_parameter_declaration_list
                         doesn't decide to commit to this parse.  */
@@ -15572,7 +15573,11 @@ cp_parser_parameter_declaration (cp_parser *parser,
 
                      cp_parser_parse_tentatively (parser);
                      cp_lexer_consume_token (parser->lexer);
+                     begin_scope (sk_function_parms, NULL_TREE);
                      cp_parser_parameter_declaration_list (parser, &error);
+                     for (t = current_binding_level->names; t; t = DECL_CHAIN (t))
+                       pop_binding (DECL_NAME (t), t);
+                     leave_scope ();
                      if (!cp_parser_error_occurred (parser) && !error)
                        done = true;
                      cp_parser_abort_tentative_parse (parser);

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 5a9f38a05507340b59793e2cf69d4866c08d5bc4..3ef4c0e2244407c59d02e05b60f3f71de2bbf22b 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2010-08-29  Jason Merrill  <jason@redhat.com>
+
+       PR c++/44991
+       * g++.dg/parse/defarg15.C: New.
+
 2010-08-29  Janus Weil  <janus@gcc.gnu.org>
 
        PR fortran/45439

diff --git a/gcc/testsuite/g++.dg/parse/defarg15.C b/gcc/testsuite/g++.dg/parse/defarg15.C
new file mode 100644 (file)
index 0000000..b93af4c
--- /dev/null
+++ b/gcc/testsuite/g++.dg/parse/defarg15.C
@@ -0,0 +1,5 @@
+// PR c++/44991
+
+class bar {
+    void foo(bool a = 3 < 2, bool b = true) {}
+};