* [PATCH] Improve wording for -freport-bug option. @ 2022-01-27 10:37 Martin Liška 2022-01-27 10:47 ` Jakub Jelinek 0 siblings, 1 reply; 6+ messages in thread From: Martin Liška @ 2022-01-27 10:37 UTC (permalink / raw) To: gcc-patches; +Cc: Richard Biener, Jakub Jelinek Hi. The patch addresses a few things related to ICE report: - 'Internal compiler error: Error reporting routines re-entered.' - unify 'internal' wording - I shortened: ''' Please submit a full bug report, with preprocessed source if appropriate. Please include the complete backtrace with any bug report. See <https://bugs.opensuse.org/> for instructions. ''' - let's recommend using -freport-bug. Now one can see: $ ./xgcc -B. ~/Programming/testcases/pr100019.C -c ... Please submit a full bug report, with preprocessed source (by using -freport-bug) and the complete backtrace. See <https://gcc.gnu.org/bugs/> for instructions. $ ./xgcc -B. ~/Programming/testcases/pr100019.C -c -freport-bug ... Please submit a full bug report, with preprocessed source and the complete backtrace. See <https://gcc.gnu.org/bugs/> for instructions. Preprocessed source stored into /tmp/ccLgekCL.out file, please attach this to your bugreport. Ready to be installed? Thanks, Martin PR web/104254 gcc/ChangeLog: * diagnostic.cc (diagnostic_initialize): Initialize report_bug flag. (diagnostic_action_after_output): Explain that -freport-bug option can be used for pre-processed file creation. Make the message shorter. (error_recursion): Rename Internal to internal. * diagnostic.h (struct diagnostic_context): New field. * opts.cc (common_handle_option): Init the field here. --- gcc/diagnostic.cc | 20 +++++++++++--------- gcc/diagnostic.h | 3 +++ gcc/opts.cc | 4 ++++ 3 files changed, 18 insertions(+), 9 deletions(-) diff --git a/gcc/diagnostic.cc b/gcc/diagnostic.cc index f5f43d5e9a7..7acdee49551 100644 --- a/gcc/diagnostic.cc +++ b/gcc/diagnostic.cc @@ -219,6 +219,8 @@ diagnostic_initialize (diagnostic_context *context, int n_opts) context->show_line_numbers_p = false; context->min_margin_width = 0; context->show_ruler_p = false; + context->report_bug = false; + if (const char *var = getenv ("GCC_EXTRA_DIAGNOSTIC_OUTPUT")) { if (!strcmp (var, "fixits-v1")) @@ -665,12 +667,12 @@ diagnostic_action_after_output (diagnostic_context *context, if (context->abort_on_error) real_abort (); - fnotice (stderr, "Please submit a full bug report,\n" - "with preprocessed source if appropriate.\n"); - if (count > 0) - fnotice (stderr, - ("Please include the complete backtrace " - "with any bug report.\n")); + const char *details + = context->report_bug ? "" : "(by using -freport-bug) "; + const char *ending = count > 0 ? "and the complete backtrace" : ""; + + fnotice (stderr, "\nPlease submit a full bug report, " + "with preprocessed source %s%s.\n", details, ending); fnotice (stderr, "See %s for instructions.\n", bug_report_url); exit (ICE_EXIT_CODE); @@ -1437,8 +1439,8 @@ num_digits (int value) /* Given a partial pathname as input, return another pathname that shares no directory elements with the pathname of __FILE__. This - is used by fancy_abort() to print `Internal compiler error in expr.cc' - instead of `Internal compiler error in ../../GCC/gcc/expr.cc'. */ + is used by fancy_abort() to print `internal compiler error in expr.cc' + instead of `internal compiler error in ../../GCC/gcc/expr.cc'. */ const char * trim_filename (const char *name) @@ -1988,7 +1990,7 @@ error_recursion (diagnostic_context *context) pp_newline_and_flush (context->printer); fnotice (stderr, - "Internal compiler error: Error reporting routines re-entered.\n"); + "internal compiler error: Error reporting routines re-entered.\n"); /* Call diagnostic_action_after_output to get the "please submit a bug report" message. */ diff --git a/gcc/diagnostic.h b/gcc/diagnostic.h index ccaa33b5817..3ca32979dde 100644 --- a/gcc/diagnostic.h +++ b/gcc/diagnostic.h @@ -340,6 +340,9 @@ struct diagnostic_context source output. */ bool show_ruler_p; + /* True if -freport-bug option is used. */ + bool report_bug; + /* Used to specify additional diagnostic output to be emitted after the rest of the diagnostic. This is for implementing -fdiagnostics-parseable-fixits and GCC_EXTRA_DIAGNOSTIC_OUTPUT. */ diff --git a/gcc/opts.cc b/gcc/opts.cc index f21c821ab2e..7d30deb8e4b 100644 --- a/gcc/opts.cc +++ b/gcc/opts.cc @@ -3158,6 +3158,10 @@ common_handle_option (struct gcc_options *opts, dc->tabstop = value; break; + case OPT_freport_bug: + dc->report_bug = value; + break; + default: /* If the flag was handled in a standard way, assume the lack of processing here is intentional. */ -- 2.34.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Improve wording for -freport-bug option. 2022-01-27 10:37 [PATCH] Improve wording for -freport-bug option Martin Liška @ 2022-01-27 10:47 ` Jakub Jelinek 2022-01-27 10:59 ` Martin Liška 0 siblings, 1 reply; 6+ messages in thread From: Jakub Jelinek @ 2022-01-27 10:47 UTC (permalink / raw) To: Martin Liška; +Cc: gcc-patches, Richard Biener On Thu, Jan 27, 2022 at 11:37:29AM +0100, Martin Liška wrote: > @@ -665,12 +667,12 @@ diagnostic_action_after_output (diagnostic_context *context, > if (context->abort_on_error) > real_abort (); > - fnotice (stderr, "Please submit a full bug report,\n" > - "with preprocessed source if appropriate.\n"); > - if (count > 0) > - fnotice (stderr, > - ("Please include the complete backtrace " > - "with any bug report.\n")); > + const char *details > + = context->report_bug ? "" : "(by using -freport-bug) "; > + const char *ending = count > 0 ? "and the complete backtrace" : ""; > + > + fnotice (stderr, "\nPlease submit a full bug report, " > + "with preprocessed source %s%s.\n", details, ending); This is highly translation unfriendly. You need to use separate full strings for all the different cases. Jakub ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Improve wording for -freport-bug option. 2022-01-27 10:47 ` Jakub Jelinek @ 2022-01-27 10:59 ` Martin Liška 2022-01-27 11:06 ` Jakub Jelinek 0 siblings, 1 reply; 6+ messages in thread From: Martin Liška @ 2022-01-27 10:59 UTC (permalink / raw) To: Jakub Jelinek; +Cc: gcc-patches, Richard Biener [-- Attachment #1: Type: text/plain, Size: 1551 bytes --] On 1/27/22 11:47, Jakub Jelinek wrote: > On Thu, Jan 27, 2022 at 11:37:29AM +0100, Martin Liška wrote: >> @@ -665,12 +667,12 @@ diagnostic_action_after_output (diagnostic_context *context, >> if (context->abort_on_error) >> real_abort (); >> - fnotice (stderr, "Please submit a full bug report,\n" >> - "with preprocessed source if appropriate.\n"); >> - if (count > 0) >> - fnotice (stderr, >> - ("Please include the complete backtrace " >> - "with any bug report.\n")); >> + const char *details >> + = context->report_bug ? "" : "(by using -freport-bug) "; >> + const char *ending = count > 0 ? "and the complete backtrace" : ""; >> + >> + fnotice (stderr, "\nPlease submit a full bug report, " >> + "with preprocessed source %s%s.\n", details, ending); > > This is highly translation unfriendly. > You need to use separate full strings for all the different cases. > > Jakub > I see, there's updated version that should be fine. $ ./xgcc -B. ~/Programming/testcases/pr100019.C -c ... Please submit a full bug report, with preprocessed source (by using -freport-bug). Please include the complete backtrace with any bug report. See <https://gcc.gnu.org/bugs/> for instructions. $ ./xgcc -B. ~/Programming/testcases/pr100019.C -c -freport-bug ... Please submit a full bug report, with preprocessed source. Please include the complete backtrace with any bug report. See <https://gcc.gnu.org/bugs/> for instructions. Preprocessed source stored into /tmp/ccReyRI7.out file, please attach this to your bugreport. Martin [-- Attachment #2: 0001-Improve-wording-for-freport-bug-option.patch --] [-- Type: text/x-patch, Size: 3897 bytes --] From a438f0b40dfbc4580ce5de2a6fba004d1fdb3930 Mon Sep 17 00:00:00 2001 From: Martin Liska <mliska@suse.cz> Date: Thu, 27 Jan 2022 11:22:42 +0100 Subject: [PATCH] Improve wording for -freport-bug option. PR web/104254 gcc/ChangeLog: * diagnostic.cc (diagnostic_initialize): Initialize report_bug flag. (diagnostic_action_after_output): Explain that -freport-bug option can be used for pre-processed file creation. Make the message shorter. (error_recursion): Rename Internal to internal. * diagnostic.h (struct diagnostic_context): New field. * opts.cc (common_handle_option): Init the field here. --- gcc/diagnostic.cc | 22 ++++++++++++++-------- gcc/diagnostic.h | 3 +++ gcc/opts.cc | 4 ++++ 3 files changed, 21 insertions(+), 8 deletions(-) diff --git a/gcc/diagnostic.cc b/gcc/diagnostic.cc index f5f43d5e9a7..29f21a16afe 100644 --- a/gcc/diagnostic.cc +++ b/gcc/diagnostic.cc @@ -219,6 +219,8 @@ diagnostic_initialize (diagnostic_context *context, int n_opts) context->show_line_numbers_p = false; context->min_margin_width = 0; context->show_ruler_p = false; + context->report_bug = false; + if (const char *var = getenv ("GCC_EXTRA_DIAGNOSTIC_OUTPUT")) { if (!strcmp (var, "fixits-v1")) @@ -665,12 +667,16 @@ diagnostic_action_after_output (diagnostic_context *context, if (context->abort_on_error) real_abort (); - fnotice (stderr, "Please submit a full bug report,\n" - "with preprocessed source if appropriate.\n"); + if (context->report_bug) + fnotice (stderr, "\nPlease submit a full bug report, " + "with preprocessed source.\n"); + else + fnotice (stderr, "\nPlease submit a full bug report, " + "with preprocessed source (by using -freport-bug).\n"); + if (count > 0) - fnotice (stderr, - ("Please include the complete backtrace " - "with any bug report.\n")); + fnotice (stderr, "Please include the complete backtrace " + "with any bug report.\n"); fnotice (stderr, "See %s for instructions.\n", bug_report_url); exit (ICE_EXIT_CODE); @@ -1437,8 +1443,8 @@ num_digits (int value) /* Given a partial pathname as input, return another pathname that shares no directory elements with the pathname of __FILE__. This - is used by fancy_abort() to print `Internal compiler error in expr.cc' - instead of `Internal compiler error in ../../GCC/gcc/expr.cc'. */ + is used by fancy_abort() to print `internal compiler error in expr.cc' + instead of `internal compiler error in ../../GCC/gcc/expr.cc'. */ const char * trim_filename (const char *name) @@ -1988,7 +1994,7 @@ error_recursion (diagnostic_context *context) pp_newline_and_flush (context->printer); fnotice (stderr, - "Internal compiler error: Error reporting routines re-entered.\n"); + "internal compiler error: Error reporting routines re-entered.\n"); /* Call diagnostic_action_after_output to get the "please submit a bug report" message. */ diff --git a/gcc/diagnostic.h b/gcc/diagnostic.h index ccaa33b5817..3ca32979dde 100644 --- a/gcc/diagnostic.h +++ b/gcc/diagnostic.h @@ -340,6 +340,9 @@ struct diagnostic_context source output. */ bool show_ruler_p; + /* True if -freport-bug option is used. */ + bool report_bug; + /* Used to specify additional diagnostic output to be emitted after the rest of the diagnostic. This is for implementing -fdiagnostics-parseable-fixits and GCC_EXTRA_DIAGNOSTIC_OUTPUT. */ diff --git a/gcc/opts.cc b/gcc/opts.cc index f21c821ab2e..7d30deb8e4b 100644 --- a/gcc/opts.cc +++ b/gcc/opts.cc @@ -3158,6 +3158,10 @@ common_handle_option (struct gcc_options *opts, dc->tabstop = value; break; + case OPT_freport_bug: + dc->report_bug = value; + break; + default: /* If the flag was handled in a standard way, assume the lack of processing here is intentional. */ -- 2.34.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Improve wording for -freport-bug option. 2022-01-27 10:59 ` Martin Liška @ 2022-01-27 11:06 ` Jakub Jelinek 2022-01-27 11:38 ` Martin Liška 0 siblings, 1 reply; 6+ messages in thread From: Jakub Jelinek @ 2022-01-27 11:06 UTC (permalink / raw) To: Martin Liška; +Cc: gcc-patches, Richard Biener On Thu, Jan 27, 2022 at 11:59:51AM +0100, Martin Liška wrote: > @@ -1988,7 +1994,7 @@ error_recursion (diagnostic_context *context) > pp_newline_and_flush (context->printer); > > fnotice (stderr, > - "Internal compiler error: Error reporting routines re-entered.\n"); > + "internal compiler error: Error reporting routines re-entered.\n"); Why do we capitalize the "E" in there? Looking for internal_error calls, I think we mostly use lower case after "internal compiler error: ", except for some cases where it should be capitalized like "SSA corruption" etc., but varasm.cc: internal_error ("Section already exists: %qs", name); looks wrong. Jakub ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Improve wording for -freport-bug option. 2022-01-27 11:06 ` Jakub Jelinek @ 2022-01-27 11:38 ` Martin Liška 2022-01-27 11:49 ` Jakub Jelinek 0 siblings, 1 reply; 6+ messages in thread From: Martin Liška @ 2022-01-27 11:38 UTC (permalink / raw) To: Jakub Jelinek; +Cc: gcc-patches, Richard Biener [-- Attachment #1: Type: text/plain, Size: 4600 bytes --] On 1/27/22 12:06, Jakub Jelinek wrote: > On Thu, Jan 27, 2022 at 11:59:51AM +0100, Martin Liška wrote: >> @@ -1988,7 +1994,7 @@ error_recursion (diagnostic_context *context) >> pp_newline_and_flush (context->printer); >> >> fnotice (stderr, >> - "Internal compiler error: Error reporting routines re-entered.\n"); >> + "internal compiler error: Error reporting routines re-entered.\n"); > > Why do we capitalize the "E" in there? That should not be capitalized. Fixed in attached patch. > Looking for internal_error calls, I think we mostly use lower case after > "internal compiler error: ", except for some cases where it should be > capitalized like "SSA corruption" etc., but > varasm.cc: internal_error ("Section already exists: %qs", name); > looks wrong. Yep, we have many more examples where a leading capital letter is used: gcc/config/cris/cris.cc: internal_error ("MULT case in %<cris_op_str%>"); gcc/config/cris/cris.h: do { if (!(x)) internal_error ("CRIS-port assertion failed: %s", #x); } while (0) gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: Last named vararg would not fit in a register"); gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: Bad register: %d", regno); gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: Bad register: %d", regno); gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: Missing %qc case in %<mmix_print_operand%>", code); gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: Bad register: %d", regno); gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: %s is not a shiftable integer", s); gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: %s is not a shiftable integer", s); gcc/config/rs6000/host-darwin.cc: internal_error ("Segmentation Fault (code)"); gcc/config/rs6000/host-darwin.cc: internal_error ("Segmentation Fault"); gcc/d/decl.cc: internal_error ("Mismatch between declaration %qE size (%wd) and " gcc/fortran/arith.cc: gfc_internal_error ("Fix min_int calculation"); gcc/fortran/data.cc: gfc_internal_error ("TODO: Vector sections in data statements"); gcc/fortran/decl.cc: gfc_internal_error ("Cannot set pointee array spec."); gcc/fortran/decl.cc: gfc_internal_error ("Missing symbol"); gcc/fortran/decl.cc: gfc_internal_error ("Cannot set Cray pointee array spec."); gcc/fortran/decl.cc: gfc_internal_error ("Failed to create structure type '%s' at %C", name); gcc/fortran/frontend-passes.cc: gfc_internal_error ("Illegal id in copy_walk_reduction_arg"); gcc/fortran/frontend-passes.cc: gfc_internal_error ("Scalarization using DIMEN_RANGE unimplemented"); gcc/fortran/interface.cc: gfc_internal_error ("Unable to find symbol %qs", sym->name); gcc/fortran/intrinsic.cc: gfc_internal_error ("Invalid standard code on intrinsic %qs (%d)", gcc/fortran/intrinsic.cc: gfc_internal_error ("Cannot convert %qs to %qs at %L", type_name, gcc/fortran/simplify.cc: gfc_internal_error ("IBITS: Bad bit"); gcc/fortran/simplify.cc: gfc_internal_error ("Reshaped array too large at %C"); gcc/fortran/simplify.cc: gfc_internal_error ("Bad type in gfc_simplify_sign"); gcc/fortran/simplify.cc: gfc_internal_error ("Failure getting length of a constant array."); gcc/fortran/simplify.cc: gfc_internal_error ("Failure getting length of a constant array."); gcc/fortran/target-memory.cc: gfc_internal_error ("Invalid expression in gfc_element_size."); gcc/fortran/target-memory.cc: gfc_internal_error ("Invalid expression in gfc_target_encode_expr."); gcc/fortran/target-memory.cc: gfc_internal_error ("Invalid expression in gfc_target_interpret_expr."); gcc/fortran/trans-intrinsic.cc: gfc_internal_error ("Intrinsic function %qs (%d) not recognized", gcc/fortran/trans-io.cc: gfc_internal_error ("Bad IO basetype (%d)", ts->type); gcc/ipa-sra.cc: internal_error ("IPA-SRA access verification failed"); gcc/ipa-sra.cc: internal_error ("Function %qs, parameter %u, has IPA-SRA accesses " gcc/ipa-sra.cc: internal_error ("Function %s, parameter %u, is used but does not " gcc/rtl.cc: internal_error ("RTL check: expected code '%s', have '%s' in %s, at %s:%d", gcc/tree-into-ssa.cc: internal_error ("SSA corruption"); gcc/tree-outof-ssa.cc: internal_error ("SSA corruption"); gcc/tree-ssa-coalesce.cc: internal_error ("SSA corruption"); gcc/varasm.cc: internal_error ("Section already exists: %qs", name); I can prepare a separate patch for next stage1 if you want? Thanks, Martin > > Jakub > [-- Attachment #2: 0001-Improve-wording-for-freport-bug-option.patch --] [-- Type: text/x-patch, Size: 3897 bytes --] From 6126a82dea18c849e0440a9f5cded8f2a7c65f14 Mon Sep 17 00:00:00 2001 From: Martin Liska <mliska@suse.cz> Date: Thu, 27 Jan 2022 11:22:42 +0100 Subject: [PATCH] Improve wording for -freport-bug option. PR web/104254 gcc/ChangeLog: * diagnostic.cc (diagnostic_initialize): Initialize report_bug flag. (diagnostic_action_after_output): Explain that -freport-bug option can be used for pre-processed file creation. Make the message shorter. (error_recursion): Rename Internal to internal. * diagnostic.h (struct diagnostic_context): New field. * opts.cc (common_handle_option): Init the field here. --- gcc/diagnostic.cc | 22 ++++++++++++++-------- gcc/diagnostic.h | 3 +++ gcc/opts.cc | 4 ++++ 3 files changed, 21 insertions(+), 8 deletions(-) diff --git a/gcc/diagnostic.cc b/gcc/diagnostic.cc index f5f43d5e9a7..10472046bdc 100644 --- a/gcc/diagnostic.cc +++ b/gcc/diagnostic.cc @@ -219,6 +219,8 @@ diagnostic_initialize (diagnostic_context *context, int n_opts) context->show_line_numbers_p = false; context->min_margin_width = 0; context->show_ruler_p = false; + context->report_bug = false; + if (const char *var = getenv ("GCC_EXTRA_DIAGNOSTIC_OUTPUT")) { if (!strcmp (var, "fixits-v1")) @@ -665,12 +667,16 @@ diagnostic_action_after_output (diagnostic_context *context, if (context->abort_on_error) real_abort (); - fnotice (stderr, "Please submit a full bug report,\n" - "with preprocessed source if appropriate.\n"); + if (context->report_bug) + fnotice (stderr, "\nPlease submit a full bug report, " + "with preprocessed source.\n"); + else + fnotice (stderr, "\nPlease submit a full bug report, " + "with preprocessed source (by using -freport-bug).\n"); + if (count > 0) - fnotice (stderr, - ("Please include the complete backtrace " - "with any bug report.\n")); + fnotice (stderr, "Please include the complete backtrace " + "with any bug report.\n"); fnotice (stderr, "See %s for instructions.\n", bug_report_url); exit (ICE_EXIT_CODE); @@ -1437,8 +1443,8 @@ num_digits (int value) /* Given a partial pathname as input, return another pathname that shares no directory elements with the pathname of __FILE__. This - is used by fancy_abort() to print `Internal compiler error in expr.cc' - instead of `Internal compiler error in ../../GCC/gcc/expr.cc'. */ + is used by fancy_abort() to print `internal compiler error in expr.cc' + instead of `internal compiler error in ../../GCC/gcc/expr.cc'. */ const char * trim_filename (const char *name) @@ -1988,7 +1994,7 @@ error_recursion (diagnostic_context *context) pp_newline_and_flush (context->printer); fnotice (stderr, - "Internal compiler error: Error reporting routines re-entered.\n"); + "internal compiler error: error reporting routines re-entered.\n"); /* Call diagnostic_action_after_output to get the "please submit a bug report" message. */ diff --git a/gcc/diagnostic.h b/gcc/diagnostic.h index ccaa33b5817..3ca32979dde 100644 --- a/gcc/diagnostic.h +++ b/gcc/diagnostic.h @@ -340,6 +340,9 @@ struct diagnostic_context source output. */ bool show_ruler_p; + /* True if -freport-bug option is used. */ + bool report_bug; + /* Used to specify additional diagnostic output to be emitted after the rest of the diagnostic. This is for implementing -fdiagnostics-parseable-fixits and GCC_EXTRA_DIAGNOSTIC_OUTPUT. */ diff --git a/gcc/opts.cc b/gcc/opts.cc index f21c821ab2e..7d30deb8e4b 100644 --- a/gcc/opts.cc +++ b/gcc/opts.cc @@ -3158,6 +3158,10 @@ common_handle_option (struct gcc_options *opts, dc->tabstop = value; break; + case OPT_freport_bug: + dc->report_bug = value; + break; + default: /* If the flag was handled in a standard way, assume the lack of processing here is intentional. */ -- 2.34.1 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] Improve wording for -freport-bug option. 2022-01-27 11:38 ` Martin Liška @ 2022-01-27 11:49 ` Jakub Jelinek 0 siblings, 0 replies; 6+ messages in thread From: Jakub Jelinek @ 2022-01-27 11:49 UTC (permalink / raw) To: Martin Liška; +Cc: gcc-patches, Richard Biener On Thu, Jan 27, 2022 at 12:38:47PM +0100, Martin Liška wrote: > Yep, we have many more examples where a leading capital letter is used: > > gcc/config/cris/cris.cc: internal_error ("MULT case in %<cris_op_str%>"); > gcc/config/cris/cris.h: do { if (!(x)) internal_error ("CRIS-port assertion failed: %s", #x); } while (0) > gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: Last named vararg would not fit in a register"); > gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: Bad register: %d", regno); > gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: Bad register: %d", regno); > gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: Missing %qc case in %<mmix_print_operand%>", code); > gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: Bad register: %d", regno); > gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: %s is not a shiftable integer", s); > gcc/config/mmix/mmix.cc: internal_error ("MMIX Internal: %s is not a shiftable integer", s); > gcc/config/rs6000/host-darwin.cc: internal_error ("Segmentation Fault (code)"); > gcc/config/rs6000/host-darwin.cc: internal_error ("Segmentation Fault"); > gcc/d/decl.cc: internal_error ("Mismatch between declaration %qE size (%wd) and " > gcc/fortran/arith.cc: gfc_internal_error ("Fix min_int calculation"); > gcc/fortran/data.cc: gfc_internal_error ("TODO: Vector sections in data statements"); > gcc/fortran/decl.cc: gfc_internal_error ("Cannot set pointee array spec."); > gcc/fortran/decl.cc: gfc_internal_error ("Missing symbol"); > gcc/fortran/decl.cc: gfc_internal_error ("Cannot set Cray pointee array spec."); > gcc/fortran/decl.cc: gfc_internal_error ("Failed to create structure type '%s' at %C", name); > gcc/fortran/frontend-passes.cc: gfc_internal_error ("Illegal id in copy_walk_reduction_arg"); > gcc/fortran/frontend-passes.cc: gfc_internal_error ("Scalarization using DIMEN_RANGE unimplemented"); > gcc/fortran/interface.cc: gfc_internal_error ("Unable to find symbol %qs", sym->name); > gcc/fortran/intrinsic.cc: gfc_internal_error ("Invalid standard code on intrinsic %qs (%d)", > gcc/fortran/intrinsic.cc: gfc_internal_error ("Cannot convert %qs to %qs at %L", type_name, > gcc/fortran/simplify.cc: gfc_internal_error ("IBITS: Bad bit"); > gcc/fortran/simplify.cc: gfc_internal_error ("Reshaped array too large at %C"); > gcc/fortran/simplify.cc: gfc_internal_error ("Bad type in gfc_simplify_sign"); > gcc/fortran/simplify.cc: gfc_internal_error ("Failure getting length of a constant array."); > gcc/fortran/simplify.cc: gfc_internal_error ("Failure getting length of a constant array."); > gcc/fortran/target-memory.cc: gfc_internal_error ("Invalid expression in gfc_element_size."); > gcc/fortran/target-memory.cc: gfc_internal_error ("Invalid expression in gfc_target_encode_expr."); > gcc/fortran/target-memory.cc: gfc_internal_error ("Invalid expression in gfc_target_interpret_expr."); > gcc/fortran/trans-intrinsic.cc: gfc_internal_error ("Intrinsic function %qs (%d) not recognized", > gcc/fortran/trans-io.cc: gfc_internal_error ("Bad IO basetype (%d)", ts->type); > gcc/ipa-sra.cc: internal_error ("IPA-SRA access verification failed"); > gcc/ipa-sra.cc: internal_error ("Function %qs, parameter %u, has IPA-SRA accesses " > gcc/ipa-sra.cc: internal_error ("Function %s, parameter %u, is used but does not " > gcc/rtl.cc: internal_error ("RTL check: expected code '%s', have '%s' in %s, at %s:%d", > gcc/tree-into-ssa.cc: internal_error ("SSA corruption"); > gcc/tree-outof-ssa.cc: internal_error ("SSA corruption"); > gcc/tree-ssa-coalesce.cc: internal_error ("SSA corruption"); > gcc/varasm.cc: internal_error ("Section already exists: %qs", name); > > I can prepare a separate patch for next stage1 if you want? Ok. MMIX, MULT, CRIS, IPA-SRA, SSA all look desirable to be capitalized. And I'd leave out gcc/fortran/, it has very different rules and I think generally capitalizes diagnostics. > PR web/104254 > > gcc/ChangeLog: > > * diagnostic.cc (diagnostic_initialize): > Initialize report_bug flag. > (diagnostic_action_after_output): > Explain that -freport-bug option can be used for pre-processed > file creation. Make the message shorter. > (error_recursion): Rename Internal to internal. > * diagnostic.h (struct diagnostic_context): New field. > * opts.cc (common_handle_option): Init the field here. Ok, thanks. Jakub ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-01-27 11:49 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-01-27 10:37 [PATCH] Improve wording for -freport-bug option Martin Liška 2022-01-27 10:47 ` Jakub Jelinek 2022-01-27 10:59 ` Martin Liška 2022-01-27 11:06 ` Jakub Jelinek 2022-01-27 11:38 ` Martin Liška 2022-01-27 11:49 ` Jakub Jelinek
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).