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 0c19a6fe47c6857f2b5f0e1d747651c33035cba6..1c3d08d693b0774e8b36060d961a4984512d952c 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 d67fa3b13d18fa741502b078e5ad9faf137670ae..591f44f49344c598154615a1117e53f39fe406aa 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 8dc87d3948de3eba032b42c9d2b6343f8dd432c1..f54ebc89446b5fb145f15779a4ac0619421dab01 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 2c2d5b2d9ef4bbb1acc0d0ecc399bdd2633f534a..a2cb30e96c9e1141321360b16215efeb5784250d 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 60dfec47a415afbde36d030d33a25ac73f12b1b9..7dded0aedd8e01e33e16e554c39035ee81d05d0f 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 4b335eb2818e19386cffdefa3eb8e54e498cc60c..7fb7c3358fc77bc88547d4cc990e94e7065be664 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 64572e5ee463e4137db8dab783504a9c893e3de8..262a1027c43afac50e6e17b4e5ddeac7bae7805e 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 af9063840982307f971d674257f5d38d32fb2a91..dbf0059a6ff8cc6cf6fa971fe18381569ec0d8b0 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 43eb01f1fcbb2930e4e65dd4ff875dfd9bfa9f5f..0d217c7bc4efb69427ade24dfb13833f52550780 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 225652cd2e43470184d3b54e9678e5f2513e1bc0..af91c89e463f28f25599958e91aaba6e8bcf716d 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 44f538e33ecb069126718b4911031370572a4d67..4a2519b3a7c216d720ec43d5023d1362de2fa698 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 e1e15504172b9c89cf1e58e45c72c065b592c8d6..c3a0bc67868bc55eacfcba88a754cfc5ee7103f0 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 ff113226cc7192ae10c0e9c41f381a014bdfc2af..959354fc912cc0f446d271cff7b795fdb1113ffe 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 8c206bbecc46049f2d5269e7cc6d326e468dff61..cf8f1e419bb3efd7d11bd3b53b48865f32ac89f8 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 05016b82b8c070f627d5fd605a871a9e9fcd0201..4b2cc07b47484f6d72279c01b56364da7a4e297c 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 42acb1b0d953ccd5a74f8bfaf5360fd3493fbec0..63203460a39fea0ca832f5daf7f9edefe9c6481f 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 248224234881a23f77dec126e08f5a08d9a86829..0a1a70349f1655227293705fec6b41eea8cdf928 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 7e326aa982746d3c966ac55daf69eb290f2f721e..bf68be1a73c0da42505f50a0b4b05e206d679dc8 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 161edad32a2a790254ef3028422a57d2e45b1d93..3cae70949db9b4619a39286aa67e6fbb4ffa338b 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 7e90dde6c8aed3434c4d5e5624a0cdceaa580d62..5994751594348ecfb450be72e4bfbe447492cc74 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 b2004b85173e24d1886bc15264b97bdded40787c..e41e15105aff3a35c206b943c984890674f0b947 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 dee0fc72212a01cea0b43770b96941170e1c31ae..8ae6e32aa50edafe2529bbcc76b54ca46fc0282e 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 e597d926f40d2032b2cc4b1c6adc4a95e38b3d77..786f9fc98c1ddd0e1f40631a4e94e555ae6ec019 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 e87ce83201d95ad877024a752ad871b8ef23ccde..15ebe24f4d78503a943e464b21df5e1e4aac675a 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 576ae9fd09bf6adfeff2aefb03955e99566927fa..53a46219a3579ec78439db5362ad9d17a3eee796 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 7c81ab4488f84412c6a93452575d0ae1cf4c9f20..aa0b5d2bed28026211e4e518ebd5c527d9c7fcc7 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 f72e277488d6659dda7cca967eeacd1d79cb48c0..72bc720bedcfaeb53c95e1a7d18089b68d419542 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 5ff3fb21929783ce1959cf02d0c933d9a56f274a..98e82c892232a640d836caf78300cb6066f97849 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 11e1063e12a76479b4a0b5e752a84cc79a82cd13..bf329248f6bfa43fc15befa6bcbf7e66689cd9c6 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 ceefee3790c56101e1c094e0065b5f168d658cdd..048cb501a579739e411dbe06534fb21c965ede10 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 44a6213a0a6f2d0f68cb0cb0cca83da6974afc2f..2901cbfe191877dcef6d786114cbcf7108cae792 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 2c9d8dd4144628e31b366d76eb1858cf2cca0e62..9be1b92413cdd1cf3217dad074044d8d30607b03 100644 (file)
@@ -4,4 +4,5 @@
 template
 <
 class
-{                              // { dg-error "" }
+{ // { dg-error "" }
+// { dg-error "-:"  }
index 544c7b219df0a6ff8677c7b35c0ce05f5bc45312..dec79aaf114faafd102a7074071eb73d6c612711 100644 (file)
@@ -2,3 +2,4 @@
 // { dg-do compile }
 
 template < int > struct S { S < // { dg-error "" }
+  // { dg-error "-:expected" }
index 80f8683a1366e591d3870f15ecc9b7693f9b050f..84b284770eeed6d6c0482264746904f752c30457 100644 (file)
@@ -1,3 +1,4 @@
 // PR c++/56534
 
 template < struct template rebind < > // { dg-error "expected|must follow" }
+// { dg-error "-:expected"  }
index cbb1221b355d0d1bcf5e6d1fd1c3f2d70caf130d..a5161d887b9f073012def26cdfac958d946efd54 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 ef9dc141bc7c4abf857ea3c7b6eda9820d001856..05f398030711d5607710c55baefa522a5c0eed5b 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 f67e0a6fa630dd74c3e46c59038ecef371d234d0..f3d87c53bcab7444d47eba6889e2a5e891d2d6fc 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 b6d19527a5ab6695237c05ba3b4d3622b8b21775..734c3f4e31aec43ec12133a0dd956fa7cab37000 100644 (file)
@@ -6,4 +6,5 @@ namespace bar
 struct foo
 {
   foo();
-};                     // { dg-error "expected" }
+};
+// { dg-error "-:expected" }