public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: Nathan Sidwell <nathan@acm.org>
To: GCC Patches <gcc-patches@gcc.gnu.org>,
	Jason Merrill <jason@redhat.com>,
	Richard Biener <richard.guenther@gmail.com>
Subject: [09/32] core diagnostics
Date: Tue, 3 Nov 2020 16:14:45 -0500	[thread overview]
Message-ID: <1fbe900e-a07d-1aa6-32f1-a36ab5b66715@acm.org> (raw)
In-Reply-To: <89819c10-e86d-9b01-5673-5223a525a135@acm.org>

[-- Attachment #1: Type: text/plain, Size: 210 bytes --]

The 'included from ...' chain that one gets at the start of a diagnostic 
needs extending to include importing.  There are a few combinations to 
handle, but nothing particularly exciting.

-- 
Nathan Sidwell


[-- Attachment #2: 09-core-diag.diff --]
[-- Type: text/x-patch, Size: 1705 bytes --]

diff --git c/gcc/diagnostic.c w/gcc/diagnostic.c
index 1b6c9845892..52bf5e0da1f 100644
--- c/gcc/diagnostic.c
+++ w/gcc/diagnostic.c
@@ -689,12 +705,13 @@ diagnostic_report_current_module (diagnostic_context *context, location_t where)
       set_last_module (context, map);
       if (! MAIN_FILE_P (map))
 	{
-	  bool first = true;
+	  bool first = true, need_inc = true, was_module = MAP_MODULE_P (map);
 	  expanded_location s = {};
 	  do
 	    {
 	      where = linemap_included_from (map);
 	      map = linemap_included_from_linemap (line_table, map);
+	      bool is_module = MAP_MODULE_P (map);
 	      s.file = LINEMAP_FILE (map);
 	      s.line = SOURCE_LINE (map, where);
 	      int col = -1;
@@ -706,14 +723,24 @@ diagnostic_report_current_module (diagnostic_context *context, location_t where)
 	      const char *line_col = maybe_line_and_column (s.line, col);
 	      static const char *const msgs[] =
 		{
-		 N_("In file included from"),
+		 NULL,
 		 N_("                 from"),
+		 N_("In file included from"),	/* 2 */
+		 N_("        included from"),
+		 N_("In module"),		/* 4 */
+		 N_("of module"),
+		 N_("In module imported at"),	/* 6 */
+		 N_("imported at"),
 		};
-	      unsigned index = !first;
+
+	      unsigned index = (was_module ? 6 : is_module ? 4
+				: need_inc ? 2 : 0) + !first;
+
 	      pp_verbatim (context->printer, "%s%s %r%s%s%R",
-			   first ? "" : ",\n", _(msgs[index]),
+			   first ? "" : was_module ? ", " : ",\n",
+			   _(msgs[index]),
 			   "locus", s.file, line_col);
-	      first = false;
+	      first = false, need_inc = was_module, was_module = is_module;
 	    }
 	  while (! MAIN_FILE_P (map));
 	  pp_verbatim (context->printer, ":");


  parent reply	other threads:[~2020-11-03 21:14 UTC|newest]

Thread overview: 68+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <7fc9e868-3db9-4972-ed67-6ff249f549c3@acm.org>
     [not found] ` <c14417b3-4286-d877-f6e2-925bfc754684@acm.org>
2020-11-03 19:39   ` [02/33] linemaps Nathan Sidwell
2020-11-03 21:13   ` [02/32] linemaps Nathan Sidwell
2020-11-06 20:10     ` Jeff Law
     [not found]   ` <0bdf00a8-d8ad-9e97-134d-6668c0e8c86b@acm.org>
2020-11-03 21:13     ` [03/32] cpp-callbacks Nathan Sidwell
2020-11-06 20:10       ` Jeff Law
     [not found]     ` <c55d12d9-e966-5b36-1538-98c9c4b418c8@acm.org>
2020-11-03 21:13       ` [04/32] cpp lexer Nathan Sidwell
2020-11-03 23:08         ` Joseph Myers
2020-11-03 23:21           ` Nathan Sidwell
2020-11-06 20:23         ` Jeff Law
2020-11-06 21:06           ` Nathan Sidwell
2020-11-09  6:15             ` Boris Kolpackov
2020-11-09 13:54             ` Nathan Sidwell
     [not found]       ` <85996243-86cf-88b4-5b3b-451eaf3a0df6@acm.org>
2020-11-03 21:14         ` [05/32] cpp files Nathan Sidwell
2020-11-17  1:27           ` Jeff Law
     [not found]         ` <ca22e13a-6869-bc48-d7c4-a04128f3fcb8@acm.org>
2020-11-03 21:14           ` [06/32] cpp macros Nathan Sidwell
2020-11-23 21:13             ` Jeff Law
     [not found]           ` <4d14b230-3263-9a13-3159-c4853f282761@acm.org>
2020-11-03 21:14             ` [07/32] cpp main Nathan Sidwell
2020-11-10 23:18               ` Jeff Law
     [not found]             ` <688bd28f-5998-0def-8c40-03b817832d63@acm.org>
2020-11-03 21:14               ` [08/32] cpp mkdeps Nathan Sidwell
2020-11-10 23:20                 ` Jeff Law
     [not found]               ` <89819c10-e86d-9b01-5673-5223a525a135@acm.org>
2020-11-03 21:14                 ` Nathan Sidwell [this message]
2020-11-21 16:57                   ` [09/32] core diagnostics Jeff Law
     [not found]                 ` <35879e15-d74a-c664-4d44-15f4b3783d77@acm.org>
2020-11-03 20:03                   ` [09/33] " Nathan Sidwell
2020-11-21 16:56                     ` Jeff Law
2020-11-03 21:14                   ` [11/32] instrumentation Nathan Sidwell
2020-11-06 20:28                     ` Jeff Law
     [not found]                   ` <9ae23c4c-67a5-a267-c939-5a96e9488612@acm.org>
2020-11-03 21:15                     ` [12/32] user documentation Nathan Sidwell
2020-11-25 19:17                       ` Jeff Law
     [not found]                     ` <c1c50ae5-44d5-4b2d-dba8-76b459b7d994@acm.org>
2020-11-03 21:15                       ` [13/32] new options Nathan Sidwell
2020-11-25 19:27                         ` Jeff Law
     [not found]                       ` <c0a0b999-f96e-177c-b393-9eaf8d1aa520@acm.org>
2020-11-03 21:15                         ` [14/32] new keywords Nathan Sidwell
2020-11-06 20:29                           ` Jeff Law
     [not found]                         ` <6666545b-0583-4812-4745-d51994465818@acm.org>
2020-11-03 21:15                           ` [15/32] new C++ lexer Nathan Sidwell
     [not found]                           ` <d6ae6a29-0da0-06a5-d86c-1b0dd4bd96e4@acm.org>
2020-11-03 21:15                             ` [16/32] new C++ infrastructure Nathan Sidwell
     [not found]                             ` <d3149435-8b5e-7c79-f4bb-80238fdfcf72@acm.org>
2020-11-03 21:15                               ` [17/32] new C++ constexpr bits Nathan Sidwell
     [not found]                               ` <1ab99df5-3997-0895-c979-f8529f476df7@acm.org>
2020-11-03 21:16                                 ` [18/32] new C++ template bits Nathan Sidwell
     [not found]                                 ` <97e9477b-7173-b7f9-a884-616b972c57ba@acm.org>
2020-11-03 21:16                                   ` [19/32] global trees Nathan Sidwell
2020-11-06 20:29                                     ` Jeff Law
     [not found]                                   ` <01f091a5-cd8b-60b6-9552-2318ecd07025@acm.org>
2020-11-03 21:16                                     ` [20/32] global constructor Nathan Sidwell
     [not found]                                     ` <a70c9177-136e-0d52-1c96-c8093588f57b@acm.org>
2020-11-03 21:16                                       ` [21/32] miscelaneous Nathan Sidwell
2020-11-05 13:30                                         ` Richard Biener
2020-11-06 17:08                                           ` Nathan Sidwell
     [not found]                                       ` <5c533ebe-440d-188e-5bdb-38c14898852c@acm.org>
2020-11-03 21:16                                         ` [22/32] miscelaneous c++ bits Nathan Sidwell
2020-11-13 13:41                                           ` Nathan Sidwell
2020-11-13 14:03                                           ` [22.2/32] module flags Nathan Sidwell
2020-11-13 14:27                                             ` Richard Biener
2020-11-13 16:25                                               ` Nathan Sidwell
     [not found]                                         ` <9e6ec23c-6b36-de70-7630-55562583696f@acm.org>
2020-11-03 21:17                                           ` [23/32] libcody Nathan Sidwell
     [not found]                                           ` <c839f3f8-9ce6-0c28-b981-d81f236a3a34@acm.org>
2020-11-03 21:17                                             ` [24/32] module mapper Nathan Sidwell
2020-11-09  6:42                                               ` Boris Kolpackov
2020-11-09 15:39                                                 ` Nathan Sidwell
2020-11-24  7:18                                                   ` Boris Kolpackov
2020-11-12 16:24                                               ` Nathan Sidwell
2020-11-30 16:18                                                 ` Jeff Law
     [not found]                                             ` <efe68976-5562-ef6a-7e86-b1daeae73670@acm.org>
2020-11-03 21:17                                               ` [25/32] modules! Nathan Sidwell
2020-11-04 12:48                                                 ` Nathan Sidwell
     [not found]                                               ` <3d138aa4-df19-df5d-54c6-ec7299749f0f@acm.org>
2020-11-03 21:17                                                 ` [26/33] name-lookup Nathan Sidwell
     [not found]                                                 ` <df8d76cf-b00f-6b4c-3b8c-132843c15d62@acm.org>
2020-11-03 21:17                                                   ` [27/32] parser Nathan Sidwell
     [not found]                                                   ` <e67492d5-3be5-0993-538d-b798875c5e89@acm.org>
2020-11-03 21:17                                                     ` [28/32] lang hook implementation Nathan Sidwell
     [not found]                                                     ` <b378cdb3-fe6f-956a-84a1-23af87ce6695@acm.org>
2020-11-03 21:18                                                       ` [29/32] make-lang.in Nathan Sidwell
     [not found]                                                       ` <bedd1f52-8a2c-b257-74bc-caf0cc1d0589@acm.org>
2020-11-03 21:18                                                         ` [30/32] test harness Nathan Sidwell
2020-11-06 20:30                                                           ` Jeff Law
     [not found]                                                         ` <0a62f316-7c23-3492-f7c3-9c1653a61a75@acm.org>
2020-11-03 21:18                                                           ` [31/32] test suite Nathan Sidwell
     [not found]                                                           ` <f4bda23c-7fa1-5a1b-e970-9fae9b2575b6@acm.org>
2020-11-03 21:18                                                             ` [32/32] fixinclude Nathan Sidwell
2020-11-04  0:04                                                               ` David Edelsohn
2020-11-18 18:36                                                                 ` Nathan Sidwell
2020-11-03 21:13 ` [01/32] langhooks Nathan Sidwell
2020-11-06 19:58   ` Jeff Law

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1fbe900e-a07d-1aa6-32f1-a36ab5b66715@acm.org \
    --to=nathan@acm.org \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=jason@redhat.com \
    --cc=richard.guenther@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).