From efc5aa6bc7c686b302c5d7fd7f65e3f6361961e1 Mon Sep 17 00:00:00 2001 From: Nathan Sidwell Date: Fri, 14 Oct 2016 20:32:03 +0000 Subject: [PATCH] diagnostic.c (diagnostic_action_after_output): Remove max error handling here .... * diagnostic.c (diagnostic_action_after_output): Remove max error handling here .... (diagnostic_report_diagnostic): ... do it here instead. testsuite/ * c-c++-common/fmax-errors.c: Make sure note is emitted. From-SVN: r241186 --- gcc/ChangeLog | 6 ++++++ gcc/diagnostic.c | 37 +++++++++++++++++++------------- gcc/testsuite/ChangeLog | 4 ++++ gcc/testsuite/c-c++-common/fmax-errors.c | 14 ++++++++++-- 4 files changed, 44 insertions(+), 17 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 2cf3f94..a131bba 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2016-10-14 Nathan Sidwell + + * diagnostic.c (diagnostic_action_after_output): Remove fatal + and max error handling here .... + (diagnostic_report_diagnostic): ... do it here instead. + 2016-10-14 David Malcolm * print-rtl-function.c (print_edge): Omit "(flags)" when none are diff --git a/gcc/diagnostic.c b/gcc/diagnostic.c index 2304e14..24aceec 100644 --- a/gcc/diagnostic.c +++ b/gcc/diagnostic.c @@ -470,18 +470,8 @@ diagnostic_action_after_output (diagnostic_context *context, diagnostic_finish (context); exit (FATAL_EXIT_CODE); } - if (context->max_errors != 0 - && ((unsigned) (diagnostic_kind_count (context, DK_ERROR) - + diagnostic_kind_count (context, DK_SORRY) - + diagnostic_kind_count (context, DK_WERROR)) - >= context->max_errors)) - { - fnotice (stderr, - "compilation terminated due to -fmax-errors=%u.\n", - context->max_errors); - diagnostic_finish (context); - exit (FATAL_EXIT_CODE); - } + /* -fmax-error handling is just before the next diagnostic is + emitted. */ break; case DK_ICE: @@ -834,9 +824,7 @@ diagnostic_report_diagnostic (diagnostic_context *context, -Wno-error=*. */ if (context->warning_as_error_requested && diagnostic->kind == DK_WARNING) - { - diagnostic->kind = DK_ERROR; - } + diagnostic->kind = DK_ERROR; if (diagnostic->option_index && diagnostic->option_index != permissive_error_option (context)) @@ -892,6 +880,25 @@ diagnostic_report_diagnostic (diagnostic_context *context, return false; } + if (diagnostic->kind != DK_NOTE && context->max_errors) + { + /* Check, before emitting the diagnostic, whether we would + exceed the limit. This way we will emit notes relevant to + the final emitted error. */ + int count = (diagnostic_kind_count (context, DK_ERROR) + + diagnostic_kind_count (context, DK_SORRY) + + diagnostic_kind_count (context, DK_WERROR)); + + if ((unsigned) count >= context->max_errors) + { + fnotice (stderr, + "compilation terminated due to -fmax-errors=%u.\n", + context->max_errors); + diagnostic_finish (context); + exit (FATAL_EXIT_CODE); + } + } + context->lock++; if (diagnostic->kind == DK_ICE || diagnostic->kind == DK_ICE_NOBT) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 849c6e4..0e3b399 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2016-10-14 Nathan Sidwell + + * c-c++-common/fmax-errors.c: Add error with note. + 2016-10-14 Jakub Jelinek PR middle-end/77959 diff --git a/gcc/testsuite/c-c++-common/fmax-errors.c b/gcc/testsuite/c-c++-common/fmax-errors.c index 1ef78eb..b44e238 100644 --- a/gcc/testsuite/c-c++-common/fmax-errors.c +++ b/gcc/testsuite/c-c++-common/fmax-errors.c @@ -1,11 +1,21 @@ /* PR c/44782 */ /* { dg-do compile } */ -/* { dg-options "-fmax-errors=3" } */ +/* { dg-options "-fmax-errors=3 -Wall" } */ void foo (unsigned int i, unsigned int j) { (i) (); /* { dg-error "" } */ (j) (); /* { dg-error "" } */ - (i+j) (); /* { dg-error "" } */ + + i + j; /* { dg-warning "" } */ + + (k) (); /* { dg-error "" } */ + /* Make sure we see the notes related to the final error we emit. */ + /* { dg-message "identifier" "" { target c } 12 } */ + + /* Warnings after the final error should not appear. */ + i + j; /* no warning. */ + (i*j) (); /* no error here due to -fmax-errors */ + } /* { dg-prune-output "compilation terminated" } */ -- 2.7.4