re PR c++/26269 (Declaring a variable too late yields bogus error message)

        PR c++/26269
        * decl.c (duplicate_decls): Return early if either
        newdecl or olddecl is error_mark_node.

        * g++.dg/other/error14.C: New test.

From-SVN: r116301

diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index 6fd125ab1d4883cd63e35f6552903bdc551a1918..366aa69f8589382152bddaa1380d4782932faefc 100644 (file)
--- a/gcc/cp/ChangeLog
+++ b/gcc/cp/ChangeLog
@@ -1,3 +1,9 @@
+2006-08-21  Lee Millward  <lee.millward@codesourcery.com>
+
+       PR c++/26269
+        * decl.c (duplicate_decls): Return early if either
+        newdecl or olddecl is error_mark_node.
+       
 2006-08-20  Mark Mitchell  <mark@codesourcery.com>
 
        PR c++/28341

diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c
index a8ec398e1ffa357950ba8378d15e4fb916e6b514..b67aea76a785c8dd0f7387ce57fc82ecad44f9a6 100644 (file)
--- a/gcc/cp/decl.c
+++ b/gcc/cp/decl.c
@@ -1124,7 +1124,7 @@ duplicate_decls (tree newdecl, tree olddecl, bool newdecl_is_friend)
      about the same declaration, so just pretend the types match here.  */
   if (TREE_TYPE (newdecl) == error_mark_node
       || TREE_TYPE (olddecl) == error_mark_node)
-    types_match = 1;
+    return error_mark_node.
 
   if (DECL_P (olddecl)
       && TREE_CODE (newdecl) == FUNCTION_DECL

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index d6e011ce9b471cd44162d67e98cb2cd5c222aa3c..d61c5ae9fd91c46124ac0643d9baf96c7712220b 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2006-08-21  Lee Millward  <lee.millward@codesourcery.com>
+
+        PR c++/26269
+        * g++.dg/other/error14.C: New test.
+       
 2006-08-21  Olivier Hainque  <hainque@adacore.com>
 
        * gnat.dg/self_aggregate_with_zeros.adb: New test.

diff --git a/gcc/testsuite/g++.dg/other/error14.C b/gcc/testsuite/g++.dg/other/error14.C
new file mode 100644 (file)
index 0000000..b3c9301
--- /dev/null
+++ b/gcc/testsuite/g++.dg/other/error14.C
@@ -0,0 +1,7 @@
+//PR c++/26269
+
+void foo()
+{
+    i;          // { dg-error "not declared in this scope" }
+    int i;
+}