OSDN Git Service

gcc/
authoraldyh <aldyh@138bc75d-0d04-0410-961f-82ee72b054a4>
Mon, 1 Jun 2009 16:09:33 +0000 (16:09 +0000)
committeraldyh <aldyh@138bc75d-0d04-0410-961f-82ee72b054a4>
Mon, 1 Jun 2009 16:09:33 +0000 (16:09 +0000)
* diagnostic.c (diagnostic_build_prefix): Always print columns.
(diagnostic_report_current_module): Print columns.
* common.opt (flag_show_column): Enable by default.
gcc/testsuite/
* lib/gcc-dg.exp (dg-bogus): Override dg-bogus.
(process-message): Expect column numbers.
* gcc.dg/va-arg-2.c: Use line 0 to indicate no column.
* gcc.dg/pch/counter-2.c: Same.
* gcc.dg/pch/valid-2.c: Same.
* gcc.dg/pch/warn-1.c: Same.
* gcc.dg/pch/valid-1.c: Same.
* gcc.dg/cpp/include2a.c: Handle lack of columns.
* gcc.dg/cpp/syshdr.c: Same.
* gcc.dg/cpp/19940712-1.c: Same.
* gcc.dg/cpp/missing-header-1.c: Same.
* gcc.dg/cpp/unc4.c: Remove -fno-show-column.
* gcc.dg/cpp/tr-warn3.c: Same.
* gcc.dg/cpp/pr29612-2.c: Same.
* gcc.dg/cpp/tr-warn4.c: Same.
* gcc.dg/cpp/Wtrigraphs.c: Same.
* gcc.dg/cpp/poison.c: Same.
* gcc.dg/cpp/arith-3.c: Same.
* gcc.dg/cpp/sysmac2.c: Same.
* gcc.dg/cpp/cpp.exp: Same.
* gcc.dg/cpp/tr-warn5.c: Same.
* gcc.dg/cpp/include2.c: Same.
* gcc.dg/cpp/Wmissingdirs.c: Same.
* gcc.dg/cpp/Wmissingdirs.c: Same.
* gcc.dg/cpp/tr-warn6.c: Same.
* gcc.dg/cpp/Wtrigraphs-2.c: Same.
* gcc.dg/cpp/macspace1.c: Same.
* gcc.dg/cpp/escape-2.c: Same.
* gcc.dg/cpp/assert2.c: Same.
* gcc.dg/cpp/undef2.c: Same.
* gcc.dg/cpp/macspace2.c: Same.
* gcc.dg/cpp/tr-warn1.c: Same.
* gcc.dg/cpp/extratokens2.c: Same.
* gcc.dg/cpp/strify2.c: Same.
* gcc.dg/cpp/Wsignprom.c: Same.
* gcc.dg/cpp/redef2.c: Same.
* gcc.dg/cpp/trad/trad.exp: Same.
* gcc.dg/cpp/arith-1.c: Same.
* gcc.dg/cpp/extratokens.c: Same.
* gcc.dg/cpp/if-mpar.c: Same.
gcc/cp/
* error.c (print_instantiation_partial_context): Print column
numbers.
libcpp/
* include/line-map.h (LAST_SOURCE_COLUMN): New.

git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@148052 138bc75d-0d04-0410-961f-82ee72b054a4

47 files changed:
gcc/ChangeLog
gcc/common.opt
gcc/cp/ChangeLog
gcc/cp/error.c
gcc/diagnostic.c
gcc/testsuite/ChangeLog
gcc/testsuite/gcc.dg/cpp/19940712-1.c
gcc/testsuite/gcc.dg/cpp/Wmissingdirs.c
gcc/testsuite/gcc.dg/cpp/Wsignprom.c
gcc/testsuite/gcc.dg/cpp/Wtrigraphs-2.c
gcc/testsuite/gcc.dg/cpp/Wtrigraphs.c
gcc/testsuite/gcc.dg/cpp/arith-1.c
gcc/testsuite/gcc.dg/cpp/arith-3.c
gcc/testsuite/gcc.dg/cpp/assert2.c
gcc/testsuite/gcc.dg/cpp/cpp.exp
gcc/testsuite/gcc.dg/cpp/escape-2.c
gcc/testsuite/gcc.dg/cpp/escape.c
gcc/testsuite/gcc.dg/cpp/extratokens.c
gcc/testsuite/gcc.dg/cpp/extratokens2.c
gcc/testsuite/gcc.dg/cpp/if-mpar.c
gcc/testsuite/gcc.dg/cpp/include2.c
gcc/testsuite/gcc.dg/cpp/include2a.c
gcc/testsuite/gcc.dg/cpp/macspace1.c
gcc/testsuite/gcc.dg/cpp/macspace2.c
gcc/testsuite/gcc.dg/cpp/missing-header-1.c
gcc/testsuite/gcc.dg/cpp/poison.c
gcc/testsuite/gcc.dg/cpp/pr29612-2.c
gcc/testsuite/gcc.dg/cpp/redef2.c
gcc/testsuite/gcc.dg/cpp/strify2.c
gcc/testsuite/gcc.dg/cpp/syshdr.c
gcc/testsuite/gcc.dg/cpp/sysmac2.c
gcc/testsuite/gcc.dg/cpp/tr-warn1.c
gcc/testsuite/gcc.dg/cpp/tr-warn3.c
gcc/testsuite/gcc.dg/cpp/tr-warn4.c
gcc/testsuite/gcc.dg/cpp/tr-warn5.c
gcc/testsuite/gcc.dg/cpp/tr-warn6.c
gcc/testsuite/gcc.dg/cpp/trad/trad.exp
gcc/testsuite/gcc.dg/cpp/unc4.c
gcc/testsuite/gcc.dg/cpp/undef2.c
gcc/testsuite/gcc.dg/pch/counter-2.c
gcc/testsuite/gcc.dg/pch/valid-1.c
gcc/testsuite/gcc.dg/pch/valid-2.c
gcc/testsuite/gcc.dg/pch/warn-1.c
gcc/testsuite/gcc.dg/va-arg-2.c
gcc/testsuite/lib/gcc-dg.exp
libcpp/ChangeLog
libcpp/include/line-map.h

index 86882e5..9789c01 100644 (file)
@@ -1,3 +1,9 @@
+2009-06-01  Aldy Hernandez  <aldyh@redhat.com>
+
+       * diagnostic.c (diagnostic_build_prefix): Always print columns.
+       (diagnostic_report_current_module): Print columns.
+       * common.opt (flag_show_column): Enable by default.
+
 2009-06-01  Luis Machado  <luisgpm@br.ibm.com>
 
        * alias.c (find_base_term): Check for NULL term before returning.
index c70639e..d3cbe16 100644 (file)
@@ -1049,8 +1049,8 @@ Common Report Var(flag_see) Init(0)
 Eliminate redundant sign extensions using LCM.
 
 fshow-column
-Common C ObjC C++ ObjC++ Report Var(flag_show_column) Init(0)
-Show column numbers in diagnostics, when available.  Default off
+Common C ObjC C++ ObjC++ Report Var(flag_show_column) Init(1)
+Show column numbers in diagnostics, when available.  Default on
 
 fsignaling-nans
 Common Report Var(flag_signaling_nans) Optimization
index e7e223e..2aee908 100644 (file)
@@ -1,3 +1,8 @@
+2009-06-01  Aldy Hernandez  <aldyh@redhat.com>
+
+       * error.c (print_instantiation_partial_context): Print column
+       numbers.
+
 2009-05-29  Ian Lance Taylor  <iant@google.com>
 
        * error.c (cp_printer): Don't use va_arg with enum type.
index 004543d..7be241d 100644 (file)
@@ -2705,19 +2705,30 @@ print_instantiation_partial_context (diagnostic_context *context,
                                     struct tinst_level *t, location_t loc)
 {
   expanded_location xloc;
+  const char *str;
   for (; ; t = t->next)
     {
       xloc = expand_location (loc);
       if (t == NULL)
        break;
-      pp_verbatim (context->printer, _("%s:%d:   instantiated from %qs\n"),
-                  xloc.file, xloc.line,
-                  decl_as_string_translate (t->decl,
-                                            TFF_DECL_SPECIFIERS | TFF_RETURN_TYPE));
+      str = decl_as_string_translate (t->decl,
+                                     TFF_DECL_SPECIFIERS | TFF_RETURN_TYPE);
+      if (flag_show_column)
+       pp_verbatim (context->printer,
+                    _("%s:%d:%d:   instantiated from %qs\n"),
+                    xloc.file, xloc.line, xloc.column, str);
+      else
+       pp_verbatim (context->printer,
+                    _("%s:%d:   instantiated from %qs\n"),
+                    xloc.file, xloc.line, str);
       loc = t->locus;
     }
-  pp_verbatim (context->printer, _("%s:%d:   instantiated from here"),
-              xloc.file, xloc.line);
+  if (flag_show_column)
+    pp_verbatim (context->printer, _("%s:%d:%d:   instantiated from here"),
+                xloc.file, xloc.line, xloc.column);
+  else
+    pp_verbatim (context->printer, _("%s:%d:   instantiated from here"),
+                xloc.file, xloc.line);
   pp_base_newline (context->printer);
 }
 
index 42a40c6..3f7bab1 100644 (file)
@@ -162,7 +162,7 @@ diagnostic_build_prefix (diagnostic_info *diagnostic)
   return
     (s.file == NULL
      ? build_message_string ("%s: %s", progname, text)
-     : flag_show_column && s.column != 0
+     : flag_show_column
      ? build_message_string ("%s:%d:%d: %s", s.file, s.line, s.column, text)
      : build_message_string ("%s:%d: %s", s.file, s.line, text));
 }
@@ -244,9 +244,15 @@ diagnostic_report_current_module (diagnostic_context *context)
       if (! MAIN_FILE_P (map))
        {
          map = INCLUDED_FROM (line_table, map);
-         pp_verbatim (context->printer,
-                      "In file included from %s:%d",
-                      map->to_file, LAST_SOURCE_LINE (map));
+         if (flag_show_column)
+           pp_verbatim (context->printer,
+                        "In file included from %s:%d:%d",
+                        map->to_file,
+                        LAST_SOURCE_LINE (map), LAST_SOURCE_COLUMN (map));
+         else
+           pp_verbatim (context->printer,
+                        "In file included from %s:%d",
+                        map->to_file, LAST_SOURCE_LINE (map));
          while (! MAIN_FILE_P (map))
            {
              map = INCLUDED_FROM (line_table, map);
index 69613d1..9e10c59 100644 (file)
@@ -1,3 +1,46 @@
+2009-06-01  Aldy Hernandez  <aldyh@redhat.com>
+
+       * lib/gcc-dg.exp (dg-bogus): Override dg-bogus.
+       (process-message): Expect column numbers.
+       * gcc.dg/va-arg-2.c: Use line 0 to indicate no column.
+       * gcc.dg/pch/counter-2.c: Same.
+       * gcc.dg/pch/valid-2.c: Same.
+       * gcc.dg/pch/warn-1.c: Same.
+       * gcc.dg/pch/valid-1.c: Same.
+       * gcc.dg/cpp/include2a.c: Handle lack of columns.
+       * gcc.dg/cpp/syshdr.c: Same.
+       * gcc.dg/cpp/19940712-1.c: Same.
+       * gcc.dg/cpp/missing-header-1.c: Same.
+       * gcc.dg/cpp/unc4.c: Remove -fno-show-column.
+       * gcc.dg/cpp/tr-warn3.c: Same.
+       * gcc.dg/cpp/pr29612-2.c: Same.
+       * gcc.dg/cpp/tr-warn4.c: Same.
+       * gcc.dg/cpp/Wtrigraphs.c: Same.
+       * gcc.dg/cpp/poison.c: Same.
+       * gcc.dg/cpp/arith-3.c: Same.
+       * gcc.dg/cpp/sysmac2.c: Same.
+       * gcc.dg/cpp/cpp.exp: Same.
+       * gcc.dg/cpp/tr-warn5.c: Same.
+       * gcc.dg/cpp/include2.c: Same.
+       * gcc.dg/cpp/Wmissingdirs.c: Same.
+       * gcc.dg/cpp/Wmissingdirs.c: Same.
+       * gcc.dg/cpp/tr-warn6.c: Same.
+       * gcc.dg/cpp/Wtrigraphs-2.c: Same.
+       * gcc.dg/cpp/macspace1.c: Same.
+       * gcc.dg/cpp/escape-2.c: Same.
+       * gcc.dg/cpp/assert2.c: Same.
+       * gcc.dg/cpp/undef2.c: Same.
+       * gcc.dg/cpp/macspace2.c: Same.
+       * gcc.dg/cpp/tr-warn1.c: Same.
+       * gcc.dg/cpp/extratokens2.c: Same.
+       * gcc.dg/cpp/strify2.c: Same.
+       * gcc.dg/cpp/Wsignprom.c: Same.
+       * gcc.dg/cpp/redef2.c: Same.
+       * gcc.dg/cpp/trad/trad.exp: Same.
+       * gcc.dg/cpp/arith-1.c: Same.
+       * gcc.dg/cpp/extratokens.c: Same.
+       * gcc.dg/cpp/if-mpar.c: Same.
+
 2009-06-01  Olivier Hainque  <hainque@adacore.com>
 
        * gnat.dg/nested_float_packed.ads: New test.
index 98bcd76..a79b2f5 100644 (file)
@@ -5,8 +5,9 @@
 /* { dg-error "unterminated comment" "" { target *-*-* } 4 } */
 /* { dg-error "unterminated comment" "header error" { target *-*-* } 8 } */
 
-#include "19940712-1.h"          /* { dg-message "" }  // In file included from: */
-#include "19940712-1a.h"  /* { dg-message "" }  // In file included from: */
+#include "19940712-1.h"
+/* { dg-message "" "In file included from:" { target *-*-* } 0 } */
+#include "19940712-1a.h"
 #include "19940712-1b.h"
 
 /* comment start in comment error
index 915aaa8..69b3aae 100644 (file)
@@ -1,5 +1,5 @@
 /* { dg-do preprocess } */
-/* { dg-options "-std=gnu99 -I /jolly/well/better/not/exist -Wmissing-include-dirs -fno-show-column" } */
+/* { dg-options "-std=gnu99 -I /jolly/well/better/not/exist -Wmissing-include-dirs" } */
 
 /* Test that -Wmissing-include-dirs issues a warning when a specified
    directory does not exist.  Source Ben Elliston, 2004-05-13.  */
index 7cdbccb..87d422b 100644 (file)
@@ -1,5 +1,5 @@
 /* { dg-do preprocess } */
-/* { dg-options "-Wall -fshow-column" } */
+/* { dg-options "-Wall" } */
 
 /* Test that -Wall emits the warnings about integer promotion changing
    the sign of an operand.  */
index 43bf134..8d61f28 100644 (file)
@@ -1,5 +1,5 @@
 /* { dg-do preprocess } */
-/* { dg-options "-std=c99 -Wtrigraphs -fno-show-column" } */
+/* { dg-options "-std=c99 -Wtrigraphs" } */
 
 /* Test warnings for trigraphs in comments, with trigraphs enabled.
    Neil Booth.  4 May 2003.  */
index 5ed6c98..d4be204 100644 (file)
@@ -1,5 +1,5 @@
 /* { dg-do preprocess } */
-/* { dg-options "-std=gnu99 -Wtrigraphs -fno-show-column" } */
+/* { dg-options "-std=gnu99 -Wtrigraphs" } */
 
 /* Test we don't double warn for trigraphs immediately after preceding
    text.  Source Neil Booth.  22 Nov 2000.  */
index 85d5848..99e3cd7 100644 (file)
@@ -7,7 +7,7 @@
    independent of target precision.  */
 
 /* { dg-do preprocess } */
-/* { dg-options -fno-show-column } */
+/* { dg-options "" } */
 
 /* Test || operator and its short circuiting.  */
 #if 0 || 0
index cc5fd1b..3015d31 100644 (file)
@@ -9,7 +9,7 @@
    Please keep changes to arith-2.c and arith-3.c in sync.  */
 
 /* { dg-do preprocess } */
-/* { dg-options "-std=c99 -fno-show-column" } */
+/* { dg-options "-std=c99" } */
 
 #include <limits.h>
 
index 130f7f5..5228bcb 100644 (file)
@@ -1,6 +1,6 @@
 /* Malformed assertion tests.  */
 /* { dg-do preprocess } */
-/* { dg-options "-fno-show-column -Wno-deprecated" } */
+/* { dg-options "-Wno-deprecated" } */
 
 #assert                /* { dg-error "without predicate" "assert w/o predicate" } */
 #assert %      /* { dg-error "an identifier" "assert punctuation" } */
index 1dc504e..acf0898 100644 (file)
@@ -37,7 +37,7 @@ dg-init
 
 # Main loop.
 dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/*.{c,S} ]] \
-       "-fno-show-column" $DEFAULT_CFLAGS
+       "" $DEFAULT_CFLAGS
 
 # All done.
 dg-finish
index e79fa91..902fad3 100644 (file)
@@ -1,7 +1,7 @@
 /* Copyright (C) 2001 Free Software Foundation, Inc.  */
 
 /* { dg-do compile } */
-/* { dg-options "-pedantic -std=c99 -fno-show-column" } */
+/* { dg-options "-pedantic -std=c99" } */
 
 /* This tests various diagnostics with -pedantic about escape
    sequences, for both the preprocessor and the compiler.
index c9dd44e..e7d5e08 100644 (file)
@@ -1,7 +1,7 @@
 /* Copyright (C) 2001 Free Software Foundation, Inc.  */
 
 /* { dg-do compile } */
-/* { dg-options "-Wtraditional -std=c89 -fno-show-column" } */
+/* { dg-options "-Wtraditional -std=c89" } */
 
 /* This tests various diagnostics with -Wtraditioanl about escape
    sequences, for both the preprocessor and the compiler.
index d3e941b..11d094a 100644 (file)
@@ -1,7 +1,7 @@
 /* Copyright (C) 2000, 2008 Free Software Foundation, Inc.  */
 
 /* { dg-do preprocess } */
-/* { dg-options "-fno-show-column -Wno-deprecated" } */
+/* { dg-options "-Wno-deprecated" } */
 
 /* Tests all directives that do not permit excess tokens at the end of
    the line.  */
index fe682bf..8e69a96 100644 (file)
@@ -1,7 +1,7 @@
 /* Copyright (C) 2002 Free Software Foundation, Inc.  */
 
 /* { dg-do preprocess } */
-/* { dg-options "-fno-show-column -Wno-endif-labels" } */
+/* { dg-options "-Wno-endif-labels" } */
 
 /* Tests that -Wno-endif-labels correctly disables the checks done by
    default (and tested in extratokens.c).  */
index 633cefc..45dd78b 100644 (file)
@@ -4,7 +4,6 @@
    missing parenthesis message.  */
 
 /* { dg-do preprocess } */
-/* { dg-options "-fshow-column" } */
 #if (1          /* { dg-error "5:missing '\\)'" "missing ')' no. 1" } */
 #endif
 
index 67f1065..de34255 100644 (file)
@@ -11,5 +11,5 @@
 
 /* These error is No such file or directory, just once.  However, this
    message is locale-dependent, so don't test for it.  */
-/* { dg-error "silly" "" { target *-*-* } 10 } */
+/* { dg-error "silly" "" { target *-*-* } 0 } */
 /* { dg-message "terminated" "" { target *-*-* } 0 } */
index 974f3f3..6a11c92 100644 (file)
@@ -11,6 +11,6 @@
 
 /* These error is No such file or directory, just once.  However, this
    message is locale-dependent, so don't test for it.  */
-/* { dg-error "silly" "" { target *-*-* } 10 } */
-/* { dg-error "missing" "" { target *-*-* } 10 } */
+/* { dg-error "silly" "" { target *-*-* } 0 } */
+/* { dg-error "missing" "" { target *-*-* } 0 } */
 /* { dg-message "terminated" "" { target *-*-* } 0 } */
index daf14d2..d857844 100644 (file)
@@ -1,6 +1,6 @@
 /* PR preprocessor/19475 */
 /* { dg-do preprocess } */
-/* { dg-options "-std=iso9899:1990 -pedantic-errors -fno-show-column" } */
+/* { dg-options "-std=iso9899:1990 -pedantic-errors" } */
 
 #define a!             /* { dg-warning "missing whitespace" } */
 #define b"             /* { dg-warning "missing whitespace" } */
index 7a81ece..1494fed 100644 (file)
@@ -1,6 +1,6 @@
 /* PR preprocessor/19475 */
 /* { dg-do preprocess } */
-/* { dg-options "-std=iso9899:1999 -pedantic-errors -fno-show-column" } */
+/* { dg-options "-std=iso9899:1999 -pedantic-errors" } */
 
 #define a!             /* { dg-error "requires whitespace" } */
 #define b"             /* { dg-error "requires whitespace" } */
index 5445d4c..dbcc2b3 100644 (file)
@@ -2,7 +2,8 @@
 /* { dg-do compile } */
 /* { dg-options "" } */
 
-#include "nonexistent.h" /* { dg-error "nonexistent.h" } */
+#include "nonexistent.h"
+/* { dg-message "nonexistent.h" "" { target *-*-* } 0 } */
 /* { dg-message "terminated" "" { target *-*-* } 0 } */
 
 /* This declaration should not receive any diagnostic.  */
index d667183..f85405c 100644 (file)
@@ -1,5 +1,4 @@
-/* { dg-do preprocess }
-   { dg-options "-fno-show-column" } */
+/* { dg-do preprocess } */
 
 #pragma GCC poison foo
 foo                    /* { dg-error "foo" "use of foo" } */
index 813eb77..fff10a8 100644 (file)
@@ -1,6 +1,6 @@
 /* PR preprocessor/29612 */
 /* { dg-do preprocess } */
-/* { dg-options "-Wtraditional -fno-show-column" } */
+/* { dg-options "-Wtraditional" } */
 
 # 6 "pr29612-2.c"
 
index 57fa3b1..1dbc100 100644 (file)
@@ -1,7 +1,7 @@
 /* Test for redefining macros with significant differences.  */
 
 /* { dg-do preprocess }
-   { dg-options "-ansi -Wall -fno-show-column" } */
+   { dg-options "-ansi -Wall" } */
 
 #define mac(a, b) (a) + (b)
 #define mac(a, b) (a) * (b)
index 2c768dc..c24220c 100644 (file)
@@ -1,7 +1,7 @@
 /* Copyright (C) 2000 Free Software Foundation, Inc.  */
 
 /* { dg-do run } */
-/* { dg-options "-std=c99 -pedantic-errors -fno-show-column" } */
+/* { dg-options "-std=c99 -pedantic-errors" } */
 
 /* Tests a whole bunch of things are correctly stringified.  */
 
index 7513773..310d5d0 100644 (file)
@@ -8,5 +8,6 @@
 /* { dg-do preprocess } */
 /* { dg-error "include_next" "good error" { target *-*-* } 4 } */
 
-#include "syshdr1.h"  /* { dg-message "" "In file included from:" } */
+#include "syshdr1.h"  
+/* { dg-message "" "In file included from:" { target *-*-* } 0 } */
 #include "syshdr2.h"
index 0d1efab..6d493a9 100644 (file)
@@ -1,7 +1,7 @@
 /* Copyright (C) 2001 Free Software Foundation, Inc.  */
 
 /* { dg-do compile } */
-/* { dg-options "-std=gnu99 -pedantic -Wtraditional -fno-show-column" } */
+/* { dg-options "-std=gnu99 -pedantic -Wtraditional" } */
 
 /* Tests diagnostics are suppressed for some macros defined in system
    headers.  */
index 259f928..37b5efe 100644 (file)
@@ -1,6 +1,6 @@
 /* Test for warnings about nontraditional directives.  */
 /* { dg-do preprocess } */
-/* { dg-options "-pedantic -Wtraditional -fno-show-column" } */
+/* { dg-options "-pedantic -Wtraditional" } */
 
 /* Block 1: K+R directives should have the # indented.  */
 
index e802b4d..3351718 100644 (file)
@@ -3,7 +3,7 @@
    warnings inside unused clauses because they are often hidden this
    way on purpose.  However they do still require indentation for K&R.  */
 /* { dg-do preprocess } */
-/* { dg-options "-pedantic -Wtraditional -fno-show-column" } */
+/* { dg-options "-pedantic -Wtraditional" } */
 
 #if 1
 
index 14dd8a3..f5b5779 100644 (file)
@@ -2,7 +2,7 @@
    Note, gcc should omit these warnings in system header files.
    By Kaveh R. Ghazi <ghazi@caip.rutgers.edu> 8/22/2000.  */
 /* { dg-do preprocess } */
-/* { dg-options "-Wtraditional -fno-show-column" } */
+/* { dg-options "-Wtraditional" } */
 
 #if 1U /* { dg-warning "traditional C rejects" "numeric constant suffix" } */
 #endif
index 16dcf4c..6867b88 100644 (file)
@@ -2,7 +2,7 @@
    Note, gcc should omit these warnings in system header files.
    By Kaveh R. Ghazi <ghazi@caip.rutgers.edu> 8/22/2000.  */
 /* { dg-do preprocess } */
-/* { dg-options "-Wtraditional -fno-show-column" } */
+/* { dg-options "-Wtraditional" } */
 
 #if +1 /* { dg-warning "unary plus operator" "unary plus operator" } */
 #endif
index e9aa851..d95409e 100644 (file)
@@ -2,7 +2,7 @@
    Note, gcc should omit these warnings in system header files.
    By Kaveh R. Ghazi <ghazi@caip.rutgers.edu> 9/8/2000.  */
 /* { dg-do preprocess } */
-/* { dg-options "-Wtraditional -fno-show-column" } */
+/* { dg-options "-Wtraditional" } */
 
 #define foo1(h) sdf "h3" fds "h" /* { dg-warning "macro argument \"h\" would be stringified" "traditional stringification" } */
 #define foo2(h2) sdf "h2" fds "h3" /* { dg-warning "macro argument \"h2\" would be stringified" "traditional stringification" } */
index 22225e5..190cfcf 100644 (file)
@@ -37,7 +37,7 @@ dg-init
 
 # Main loop.
 dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/*.\[cS\]]] \
-       "-fno-show-column" $DEFAULT_TRADCPPFLAGS
+       "" $DEFAULT_TRADCPPFLAGS
 
 # All done.
 dg-finish
index 758e5bb..10c49e9 100644 (file)
@@ -1,5 +1,4 @@
 /* { dg-do preprocess } */
-/* { dg-options "-fno-show-column" } */
 
 /* Tests for un-terminated conditional diagnostics.
    Copyright (c) 1999 Free Software Foundation.
index 4e6a690..5614e03 100644 (file)
@@ -1,9 +1,7 @@
 /* C99 6.10.8 para 4: None of [the predefined macro names] shall be
-   the subject of a #define or an #undef preprocessing directive.  We
-   pass -fno-show-column as otherwise dejagnu gets confused.  */
+   the subject of a #define or an #undef preprocessing directive.  */
 
 /* { dg-do preprocess } */
-/* { dg-options "-fno-show-column" } */
 
 #undef __DATE__                /* { dg-warning "undefining" "__DATE__" } */
 #undef __TIME__                /* { dg-warning "undefining" "__TIME__" } */
index 6dd2245..14ce249 100644 (file)
@@ -8,7 +8,7 @@
 #endif
 
 #include "counter-2.h" /* { dg-warning "not used because `__COUNTER__' is invalid" } */
-/* { dg-error "counter-2.h: No such file or directory" "no such file" { target *-*-* } 10 } */
+/* { dg-error "counter-2.h: No such file or directory" "no such file" { target *-*-* } 0 } */
 /* { dg-error "one or more PCH files were found, but they were invalid" "invalid files" { target *-*-* } 10 } */
 /* { dg-message "terminated" "" { target *-*-* } 0 } */
 
index 3ee9091..b7f22d0 100644 (file)
@@ -1,8 +1,8 @@
 /* { dg-options "-I. -Winvalid-pch -g" } */
 
 #include "valid-1.h"/* { dg-warning "created with -gnone, but used with -g" } */
-/* { dg-error "No such file" "no such file" { target *-*-* } 3 } */
-/* { dg-error "they were invalid" "invalid files" { target *-*-* } 3 } */
+/* { dg-error "No such file" "no such file" { target *-*-* } 0 } */
+/* { dg-error "they were invalid" "invalid files" { target *-*-* } 0 } */
 /* { dg-message "terminated" "" { target *-*-* } 0 } */
 
 int x;
index 34269a8..3d8cb14 100644 (file)
@@ -1,7 +1,7 @@
 /* { dg-options "-I. -Winvalid-pch -fexceptions" } */
 
 #include "valid-2.h" /* { dg-warning "settings for -fexceptions do not match" } */
-/* { dg-error "No such file" "no such file" { target *-*-* } 3 } */
-/* { dg-error "they were invalid" "invalid files" { target *-*-* } 3 } */
+/* { dg-error "No such file" "no such file" { target *-*-* } 0 } */
+/* { dg-error "they were invalid" "invalid files" { target *-*-* } 0 } */
 /* { dg-message "terminated" "" { target *-*-* } 0 } */
 int x;
index 64944c7..1d31cef 100644 (file)
@@ -3,8 +3,8 @@
 #define DEFINED_VALUE 3
 
 #include "warn-1.h"/* { dg-warning "not used because .DEFINED_VALUE. is defined" } */
-/* { dg-error "No such file" "no such file" { target *-*-* } 5 } */
-/* { dg-error "they were invalid" "invalid files" { target *-*-* } 5 } */
+/* { dg-error "No such file" "no such file" { target *-*-* } 0 } */
+/* { dg-error "they were invalid" "invalid files" { target *-*-* } 0 } */
 /* { dg-message "terminated" "" { target *-*-* } 0 } */
 
 
index 2fd0ed9..e1c915f 100644 (file)
@@ -5,7 +5,7 @@
 
 #include <varargs.h>  /* { dg-bogus "varargs.h" "missing file" } */
 
-/* { dg-message "" "In file included from" { target *-*-* } 6 } */
+/* { dg-message "file included from" "file included from" { target *-*-* } 0 } */
 /* { dg-error "no longer implements" "#error 1" { target *-*-* } 4 } */
 /* { dg-error "Revise your code" "#error 2" { target *-*-* } 5 } */
 
index 7d00acf..e906265 100644 (file)
@@ -620,6 +620,17 @@ if { [info procs saved-dg-error] == [list] \
 
        process-message saved-dg-error "$gcc_error_prefix" "$args"
     }
+
+    # Override dg-bogus at the same time.  It doesn't handle a prefix
+    # but its expression should include a column number.  Otherwise the
+    # line number can match the column number for other messages, leading
+    # to insanity.
+    rename dg-bogus saved-dg-bogus
+
+    proc dg-bogus { args } {
+       upvar dg-messages dg-messages
+       process-message saved-dg-bogus "" $args
+    }
 }
 
 # Modify the regular expression saved by a DejaGnu message directive to
@@ -641,20 +652,26 @@ proc process-message { msgproc msgprefix dgargs } {
        return;
     }
 
-    # Prepend the message prefix to the regular expression and make
-    # it match a single line.
+    # Get the entry for the new message.  Prepend the message prefix to
+    # the regular expression and make it match a single line.
     set newentry [lindex ${dg-messages} end]
     set expmsg [lindex $newentry 2]
 
-    # If we have a column...
+    # Handle column numbers from the specified expression (if there is
+    # one) and set up the search expression that will be used by DejaGnu.
     if [regexp "^(\[0-9\]+):" $expmsg "" column] {
-       # Remove "COLUMN:"
+       # The expression in the directive included a column number.
+       # Remove "COLUMN:" from the original expression and move it
+       # to the proper place in the search expression.
        regsub "^\[0-9\]+:" $expmsg "" expmsg
-
-       # Include the column in the search expression.
-       set expmsg "$column: $msgprefix\[^\n]*$expmsg"
+       set expmsg "$column: $msgprefix\[^\n\]*$expmsg"
+    } elseif [string match "" [lindex $newentry 0]] {
+       # The specified line number is 0; don't expect a column number.
+       set expmsg "$msgprefix\[^\n\]*$expmsg"
     } else {
-       set expmsg "$msgprefix\[^\n]*$expmsg"
+       # There is no column number in the search expression, but we
+       # should expect one in the message itself.
+       set expmsg "\[0-9\]+: $msgprefix\[^\n\]*$expmsg"
     }
 
     set newentry [lreplace $newentry 2 2 $expmsg]
index f864528..7023582 100644 (file)
@@ -1,3 +1,7 @@
+2009-06-01  Aldy Hernandez  <aldyh@redhat.com>
+
+       * include/line-map.h (LAST_SOURCE_COLUMN): New.
+
 2009-06-01  Ian Lance Taylor  <iant@google.com>
 
        * include/cpp-id-data.h: Add extern "C".
index 962525f..09e72f1 100644 (file)
@@ -158,6 +158,8 @@ extern const struct line_map *linemap_lookup
    of the #include, or other directive, that caused a map change.  */
 #define LAST_SOURCE_LINE(MAP) \
   SOURCE_LINE (MAP, LAST_SOURCE_LINE_LOCATION (MAP))
+#define LAST_SOURCE_COLUMN(MAP) \
+  SOURCE_COLUMN (MAP, LAST_SOURCE_LINE_LOCATION (MAP))
 #define LAST_SOURCE_LINE_LOCATION(MAP) \
   ((((MAP)[1].start_location - 1 - (MAP)->start_location) \
     & ~((1 << (MAP)->column_bits) - 1))                          \