EOF has a location
authorNathan Sidwell <nathan@acm.org>
Fri, 8 May 2020 15:43:16 +0000 (08:43 -0700)
committerNathan Sidwell <nathan@acm.org>
Fri, 8 May 2020 15:47:13 +0000 (08:47 -0700)
There's no need to special-case EOF's location.  For the complete file we
give it a legitimate location.  And for deferred parses we now zap a temporary EOF
onto the next token, so we can just use its location anyway.

gcc/cp/
* parser.c (cp_lexer_set_source_position_from_token): EOF has a
location too.

gcc/testsuite/
* c-c++-common/raw-string-6.c: Adjust EOF error location.
* g++.dg/cpp0x/decltype63.C: Likewise.
* g++.dg/cpp0x/gen-attrs-64.C: Likewise.
* g++.dg/cpp0x/pr68726.C: Likewise.
* g++.dg/cpp0x/pr78341.C: Likewise.
* g++.dg/cpp1y/pr65202.C: Likewise.
* g++.dg/cpp1z/class-deduction44.C: Likewise.
* g++.dg/diagnostic/unclosed-extern-c.C: Likewise.
* g++.dg/diagnostic/unclosed-function.C: Likewise.
* g++.dg/diagnostic/unclosed-namespace.C: Likewise.
* g++.dg/diagnostic/unclosed-struct.C: Likewise.
* g++.dg/ext/pr84598.C: Likewise.
* g++.dg/other/switch4.C: Likewise.
* g++.dg/parse/crash10.C: Likewise.
* g++.dg/parse/crash18.C: Likewise.
* g++.dg/parse/crash35.C: Likewise.
* g++.dg/parse/crash59.C: Likewise.
* g++.dg/parse/crash61.C: Likewise.
* g++.dg/parse/crash67.C: Likewise.
* g++.dg/parse/ctor3.C: Likewise.
* g++.dg/parse/error14.C: Likewise.
* g++.dg/parse/error5.C: Likewise.
* g++.dg/parse/error56.C: Likewise.
* g++.dg/parse/invalid1.C: Likewise.
* g++.dg/parse/parameter-declaration-1.C: Likewise.
* g++.dg/parse/parser-pr28152-2.C: Likewise.
* g++.dg/parse/parser-pr28152.C: Likewise.
* g++.dg/parse/pr68722.C: Likewise.
* g++.dg/pr46852.C: Likewise.
* g++.dg/pr46868.C: Likewise.
* g++.dg/template/crash115.C: Likewise.
* g++.dg/template/crash43.C: Likewise.
* g++.dg/template/error-recovery1.C: Likewise.
* g++.dg/template/error57.C: Likewise.
* g++.old-deja/g++.other/crash31.C: Likewise.

38 files changed:
gcc/cp/ChangeLog
gcc/cp/parser.c
gcc/testsuite/ChangeLog
gcc/testsuite/c-c++-common/raw-string-6.c
gcc/testsuite/g++.dg/cpp0x/decltype63.C
gcc/testsuite/g++.dg/cpp0x/gen-attrs-64.C
gcc/testsuite/g++.dg/cpp0x/pr68726.C
gcc/testsuite/g++.dg/cpp0x/pr78341.C
gcc/testsuite/g++.dg/cpp1y/pr65202.C
gcc/testsuite/g++.dg/cpp1z/class-deduction44.C
gcc/testsuite/g++.dg/diagnostic/unclosed-extern-c.C
gcc/testsuite/g++.dg/diagnostic/unclosed-function.C
gcc/testsuite/g++.dg/diagnostic/unclosed-namespace.C
gcc/testsuite/g++.dg/diagnostic/unclosed-struct.C
gcc/testsuite/g++.dg/ext/pr84598.C
gcc/testsuite/g++.dg/other/switch4.C
gcc/testsuite/g++.dg/parse/crash10.C
gcc/testsuite/g++.dg/parse/crash18.C
gcc/testsuite/g++.dg/parse/crash35.C
gcc/testsuite/g++.dg/parse/crash59.C
gcc/testsuite/g++.dg/parse/crash61.C
gcc/testsuite/g++.dg/parse/crash67.C
gcc/testsuite/g++.dg/parse/ctor3.C
gcc/testsuite/g++.dg/parse/error14.C
gcc/testsuite/g++.dg/parse/error5.C
gcc/testsuite/g++.dg/parse/error56.C
gcc/testsuite/g++.dg/parse/invalid1.C
gcc/testsuite/g++.dg/parse/parameter-declaration-1.C
gcc/testsuite/g++.dg/parse/parser-pr28152-2.C
gcc/testsuite/g++.dg/parse/parser-pr28152.C
gcc/testsuite/g++.dg/parse/pr68722.C
gcc/testsuite/g++.dg/pr46852.C
gcc/testsuite/g++.dg/pr46868.C
gcc/testsuite/g++.dg/template/crash115.C
gcc/testsuite/g++.dg/template/crash43.C
gcc/testsuite/g++.dg/template/error-recovery1.C
gcc/testsuite/g++.dg/template/error57.C
gcc/testsuite/g++.old-deja/g++.other/crash31.C

index 0c19a6f..1c3d08d 100644 (file)
@@ -1,3 +1,8 @@
+2020-05-08  Nathan Sidwell  <nathan@acm.org>
+
+       * parser.c (cp_lexer_set_source_position_from_token): EOF has a
+       location too.
+
 2020-05-07  Iain Sandoe  <iain@sandoe.co.uk>
 
        PR c++/94817
index d67fa3b..591f44f 100644 (file)
@@ -895,10 +895,7 @@ cp_lexer_get_preprocessor_token (cp_lexer *lexer, cp_token *token)
 static inline void
 cp_lexer_set_source_position_from_token (cp_token *token)
 {
-  if (token->type != CPP_EOF)
-    {
-      input_location = token->location;
-    }
+  input_location = token->location;
 }
 
 /* Update the globals input_location and the input file stack from LEXER.  */
index 8dc87d3..f54ebc8 100644 (file)
@@ -1,3 +1,41 @@
+2020-05-08  Nathan Sidwell  <nathan@acm.org>
+
+       * c-c++-common/raw-string-6.c: Adjust EOF error location.
+       * g++.dg/cpp0x/decltype63.C: Likewise.
+       * g++.dg/cpp0x/gen-attrs-64.C: Likewise.
+       * g++.dg/cpp0x/pr68726.C: Likewise.
+       * g++.dg/cpp0x/pr78341.C: Likewise.
+       * g++.dg/cpp1y/pr65202.C: Likewise.
+       * g++.dg/cpp1z/class-deduction44.C: Likewise.
+       * g++.dg/diagnostic/unclosed-extern-c.C: Likewise.
+       * g++.dg/diagnostic/unclosed-function.C: Likewise.
+       * g++.dg/diagnostic/unclosed-namespace.C: Likewise.
+       * g++.dg/diagnostic/unclosed-struct.C: Likewise.
+       * g++.dg/ext/pr84598.C: Likewise.
+       * g++.dg/other/switch4.C: Likewise.
+       * g++.dg/parse/crash10.C: Likewise.
+       * g++.dg/parse/crash18.C: Likewise.
+       * g++.dg/parse/crash35.C: Likewise.
+       * g++.dg/parse/crash59.C: Likewise.
+       * g++.dg/parse/crash61.C: Likewise.
+       * g++.dg/parse/crash67.C: Likewise.
+       * g++.dg/parse/ctor3.C: Likewise.
+       * g++.dg/parse/error14.C: Likewise.
+       * g++.dg/parse/error5.C: Likewise.
+       * g++.dg/parse/error56.C: Likewise.
+       * g++.dg/parse/invalid1.C: Likewise.
+       * g++.dg/parse/parameter-declaration-1.C: Likewise.
+       * g++.dg/parse/parser-pr28152-2.C: Likewise.
+       * g++.dg/parse/parser-pr28152.C: Likewise.
+       * g++.dg/parse/pr68722.C: Likewise.
+       * g++.dg/pr46852.C: Likewise.
+       * g++.dg/pr46868.C: Likewise.
+       * g++.dg/template/crash115.C: Likewise.
+       * g++.dg/template/crash43.C: Likewise.
+       * g++.dg/template/error-recovery1.C: Likewise.
+       * g++.dg/template/error57.C: Likewise.
+       * g++.old-deja/g++.other/crash31.C: Likewise.
+
 2020-05-08  Eric Botcazou  <ebotcazou@adacore.com>
 
        * gnat.dg/addr15.adb: New test.
index 2c2d5b2..a2cb30e 100644 (file)
@@ -2,5 +2,6 @@
 // { dg-options "-std=gnu99" { target c } }
 // { dg-options "-std=c++0x" { target c++ } }
 
-const void *s0 = R"ouch()ouCh";        // { dg-error "at end of input" "end" }
-       // { dg-error "unterminated raw string" "unterminated" { target *-*-* } .-1 }
+const void *s0 = R"ouch()ouCh";        // { dg-error "unterminated raw string" "unterminated" }
+// { dg-error "at end of input" "end" { target c } .-1 }
+// { dg-error "-:at end of input" "end" { target c++ } .+1 }
index 60dfec4..7dded0a 100644 (file)
@@ -4,4 +4,5 @@
 template<typename T>
 class C{
  T t;
- decltype(t)::a::  // { dg-error "expected" }
+  decltype(t)::a:: // { dg-error "expected" }
+// { dg-error "-:expected"  }
index 4b335eb..7fb7c33 100644 (file)
@@ -1,4 +1,5 @@
 // PR c++/85140
 // { dg-do compile { target c++11 } }
 
-namespace alignas() N  {}      // { dg-error "expected" }
+namespace alignas() N  {} // { dg-error "expected"  }
+// { dg-error "-:expected"  }
index 64572e5..262a102 100644 (file)
@@ -4,4 +4,5 @@ template <typename> struct A {
   template <typename, > struct __construct_helper;  // { dg-error "expected" }
   template <typename... _Args>
   using __has_construct typename __construct_helper<_Args...>::type;  // { dg-error "expected" }
-} struct : A<int> {  // { dg-error "expected" }
+} struct : A<int> {   // { dg-error "expected" }
+  // { dg-error "-:expected" }
index af90638..dbf0059 100644 (file)
@@ -1,4 +1,5 @@
 // PR c++/78341
 // { dg-do compile { target c++11 } }
 
-alignas (alignas double // { dg-error "" }
+alignas (alignas double // { dg-error "expected" }
+// { dg-error "-:expected" }
index 43eb01f..0d217c7 100644 (file)
@@ -24,4 +24,5 @@ int main()
 {
     foo<ns::bar> f;
     adl::swap(f, f) // { dg-error "expected ';'" }
-} // { dg-error "expected '.'" "expected end of namespace" }
+}
+// { dg-error "-:expected '.'" "expected end of namespace" }
index 225652c..af91c89 100644 (file)
@@ -2,4 +2,5 @@
 // { dg-do compile { target c++17 } }
 
 template <typename> struct A;
-template <typename> struct B : A < B { ,  // { dg-error "" }
+template <typename> struct B : A < B { , // { dg-error ""  }
+// { dg-error "-:expected"  }
index 44f538e..4a2519b 100644 (file)
@@ -8,5 +8,6 @@ template <typename T> // { dg-error "template with C linkage" }
 void bar (void);
 // { dg-message "1: 'extern .C.' linkage started here" "" { target *-*-* } open_extern_c }
 
-void test (void); /* { dg-error "17: expected '.' at end of input" } */
+void test (void);
 // { message "12: to match this '.'" "" { target *-*-* } open_extern_c }
+/* { dg-error "-:expected '.' at end of input"  } */
index e1e1550..c3a0bc6 100644 (file)
@@ -1,3 +1,4 @@
 void test (void)
 { /* { dg-message "1: to match this '.'" } */
-  int filler; /* { dg-error "13: expected '.' at end of input" } */
+  int filler;
+  /* { dg-error "-:expected '.' at end of input" } */
index ff11322..959354f 100644 (file)
@@ -1,2 +1,3 @@
 namespace unclosed { /* { dg-message "20: to match this '.'" } */
-int filler; /* { dg-error "11: expected '.' at end of input" } */
+int filler;
+/* { dg-error "-:expected '.' at end of input" } */
index 8c206bb..cf8f1e4 100644 (file)
@@ -1,3 +1,3 @@
 struct unclosed { /* { dg-message "17: to match this '.'" } */
-  int dummy; /* { dg-error "12: expected '.' at end of input" } */
-  // { dg-error "expected unqualified-id at end of input" "" { target *-*-* } .-1 }
+  int dummy;
+  // { dg-error "-:expected"  }
index 05016b8..4b2cc07 100644 (file)
@@ -1,3 +1,4 @@
 // { dg-options "-fpermissive -w" }
 
-template<int b> __attribute__ a([] { class c, __attribute__(vector_size(operator+()))) d;  // { dg-error "expected" }
+template<int b> __attribute__ a([] { class c, __attribute__(vector_size(operator+()))) d; // { dg-error "" }
+// { dg-error "-:expected" }
index 42acb1b..6320346 100644 (file)
@@ -4,3 +4,4 @@ class a b;  // { dg-error "aggregate" }
 void c() {
   switch ()  // { dg-error "expected" }
   case b  // { dg-error "expected" }
+    // { dg-error "-:expected" }
index 2482242..0a1a703 100644 (file)
@@ -5,9 +5,8 @@
 
 // PR c++ 10953. ICE
 
-// { dg-bogus "" "" { target *-*-* } 14 }
-
 class 
 {
   typename::
-; // { dg-error "" }
+; // { dg-error "expected" }
+ // { dg-error "-:expected" }
index 7e326aa..bf68be1 100644 (file)
@@ -4,4 +4,4 @@
 // PR 18095: ICE
 // Origin: Volker Reichelt <reichelt@gcc.gnu.org>
 
-struct A // { dg-error "expected" }
+struct A // { dg-error "-:expected" }
index 161edad..3cae709 100644 (file)
@@ -4,4 +4,4 @@
 struct a {};
 
 class foo : public a, a                // { dg-error "duplicate base" }
-{ /* { dg-error "at end of input" } */
+{ // { dg-error "-:at end of input" } 
index 7e90dde..5994751 100644 (file)
@@ -1,4 +1,4 @@
 // PR c++/53003
 
 struct A{ void a{} return b  // { dg-error "16:function definition" }
-// { dg-error "expected" "" { target *-*-* } .-1 }
+// { dg-error "-:expected" }
index b2004b8..e41e151 100644 (file)
@@ -5,3 +5,4 @@
 struct pair { constexpr pair (const) : }; // { dg-error "" }
 template <0> make_pair () {}             // { dg-error "" }
 pair prefix[] = { 0, make_pair }         // { dg-error "" }
+// { dg-error "-:expected" "" { target c++11 } }
index dee0fc7..8ae6e32 100644 (file)
@@ -3,4 +3,5 @@
 
 class x0;
 template <x1> x2() {  // { dg-error "declared|type" }
-x0 x3 = x3.  // { dg-error "expected|incomplete type" }
+x0 x3 = x3. // { dg-error "incomplete type" }
+// { dg-error "-:expected" }
index e597d92..786f9fc 100644 (file)
@@ -4,5 +4,5 @@ struct A {};
 
 struct B : A
 {
-  B() : A {}                   // { dg-error "initializer|expected" }
-};
+  B() : A {} // { dg-error "initializer" "" { target c++98_only } }
+};                     // { dg-error "expected" }
index e87ce83..15ebe24 100644 (file)
@@ -19,6 +19,5 @@ struct X
         Zinc<int>( //);
                  //    }
 
-}; // { dg-error "2:expected '.' at end of input" "at end of input" }
-   // { dg-error "1:expected primary-expression before '.' token" "primary" { target *-*-* } .-1 }
-   // { dg-error "1:expected unqualified-id" "unqualified-id" { target *-*-* } .-2 }
+};   // { dg-error "1:expected primary-expression before '.' token" "primary"  }
+// { dg-error "-:expected" "at end of input" }
index 576ae9f..53a4621 100644 (file)
@@ -12,7 +12,7 @@ class Foo { int foo() return 0; } };
 // message has been generated by cp_parser_error() which does not
 // necessarily allow accurate column number display. At some point, we will
 // need make cp_parser_error() report more accurate column numbers.
-// { dg-error "30:expected '\{' at end of input" "brace" { target *-*-* } 5 }
+// { dg-error "31:expected '\{' at end of input" "brace" { target *-*-* } 5 }
 
 // { dg-error "34:expected ';' after class definition" "semicolon" {target *-*-* } 5 }
 
index 7c81ab4..aa0b5d2 100644 (file)
@@ -2,4 +2,6 @@
 
 template <0> int __copy_streambufs_eof; // { dg-error "" }
 class {
+// { dg-error "forbids" "" { target *-*-* } .+1 }
     friend __copy_streambufs_eof <> ( // { dg-error "" }
+ // { dg-error "-:expected" }
index f72e277..72bc720 100644 (file)
@@ -2,4 +2,5 @@
 // { dg-do compile { target c++11 } }
 // { dg-options "-w" }
 
-int foo ( foo += *[ // { dg-error "" }
+int foo ( foo += *[ // { dg-error "expected" }
+// { dg-error "-:invalid|expected" }
index 5ff3fb2..98e82c8 100644 (file)
@@ -3,4 +3,5 @@
 // { dg-do compile }
 
 struct {
-   a(void = 0; a(0), a(0)      // { dg-error "" }
+  a(void = 0; a(0), a(0) // { dg-error "" }
+// { dg-error "-:expected" }
index 11e1063..bf32924 100644 (file)
@@ -8,4 +8,4 @@ main (void)
 
   z = __complex__ (1.90000007326203904e+19, 0.0);   // { dg-error "expected primary-expression before '__complex__'" "primary-expression" } 
   z = __complex__ (1.0e+0, 0.0) / z;    // { dg-error "expected primary-expression before '__complex__'" "primaty-expression" } 
-  // { dg-error "at end of input" "end" { target *-*-* } .-1 }
+  // { dg-error "-:at end of input" "end" }
index ceefee3..048cb50 100644 (file)
@@ -8,4 +8,4 @@ main (void)
 
   z = _Complex (1.90000007326203904e+19, 0.0);   // { dg-error "expected primary-expression before '_Complex'" "primary-expression" } 
   z = _Complex (1.0e+0, 0.0) / z;    // { dg-error "expected primary-expression before '_Complex'" "primary-expression" } 
-  // { dg-error "at end of input" "end" { target *-*-* } .-1 }
+  // { dg-error "-:at end of input" "end" }
index 44a6213..2901cbf 100644 (file)
@@ -4,6 +4,6 @@ class A {
   &__loc   // { dg-error "" }
 } class ios_base {  // { dg-error "" }
   A _M_ios_locale ios_base(ios_base &) template <_Traits> class basic_ios {  // { dg-error "" }
-basic_ios basic_ios = operator=  // { dg-error "" }
-
+    basic_ios basic_ios = operator= // { dg-error "" }
 // { dg-prune-output "file ends in default argument" }
+// { dg-error "-:expected" }
index 2c9d8dd..9be1b92 100644 (file)
@@ -4,4 +4,5 @@
 template
 <
 class
-{                              // { dg-error "" }
+{ // { dg-error "" }
+// { dg-error "-:"  }
index 544c7b2..dec79aa 100644 (file)
@@ -2,3 +2,4 @@
 // { dg-do compile }
 
 template < int > struct S { S < // { dg-error "" }
+  // { dg-error "-:expected" }
index 80f8683..84b2847 100644 (file)
@@ -1,3 +1,4 @@
 // PR c++/56534
 
 template < struct template rebind < > // { dg-error "expected|must follow" }
+// { dg-error "-:expected"  }
index cbb1221..a5161d8 100644 (file)
@@ -4,5 +4,5 @@ extern "C" {
   template<typename _Tp>  // { dg-error "C" }   
   struct ___is_pod {
     enum {
-      __value = (sizeof(__gnu_internal::__test_type<_Tp>(0)))}; // { dg-error "declared|expected" }
-
+      __value = (sizeof(__gnu_internal::__test_type<_Tp>(0)))}; // { dg-error "expected|declared"  }
+// { dg-error "-:expected"  }
index ef9dc14..05f3980 100644 (file)
@@ -6,4 +6,5 @@ template < typename > struct T
   template < typename >
   template < bool > struct T < S > // { dg-error "" }
   {
-    void f () {                        // { dg-error "" }
+    void f () { // { dg-error "expected" }
+// { dg-error "-:expected"  }
index f67e0a6..f3d87c5 100644 (file)
@@ -2,4 +2,4 @@
 
 template <int()> struct a;
 template <int(b)> a < b                // { dg-error "int" }
-// { dg-error "expected" "" { target *-*-* } .-1 }
+// { dg-error "-:expected"  }
index b6d1952..734c3f4 100644 (file)
@@ -6,4 +6,5 @@ namespace bar
 struct foo
 {
   foo();
-};                     // { dg-error "expected" }
+};
+// { dg-error "-:expected" }