On Wed, 2020-01-08 at 04:02 -0500, David Malcolm wrote: > I may be able to self-approve this. It's used by the diagnostic_path > patch, and by the analyzer test suite. Perhaps better to make > undocumeted, or do it via a DejaGnu pruning directive, but I wanted > to get v5 of the kit posted. > > This patch implements -fdiagnostics-nn-line-numbers, a new option > which makes diagnostic_show_locus print "NN" rather than specific > line numbers when printing the left margin. > > This is intended purely to make it easier to write certain kinds of > DejaGnu test; various integration tests for diagnostic paths later > in the patch kit make use of it. > > gcc/ChangeLog: > * common.opt (fdiagnostics-nn-line-numbers): New option. > * diagnostic-show-locus.c (layout::m_use_nn_for_line_numbers_p): > New field. > (layout::layout): Initialize it. > (layout::calculate_linenum_width): Use it when computing > m_linenum_width. > (layout::print_source_line): Implement printing "NN" rather than > the line number. > (selftest::test_line_numbers_multiline_range): Add a test of "NN" > printing. > * diagnostic.c (diagnostic_initialize): Initialize > use_nn_for_line_numbers_p. > (num_digits): Add "use_nn_p" param. > (selftest::test_num_digits): Add a test for use_nn_p==true. > * diagnostic.h (diagnostic_context::use_nn_for_line_numbers_p): > New field. > (num_digits): Add optional "use_nn_p" param. > * doc/invoke.texi (-fdiagnostics-nn-line-numbers): New option. > * dwarf2out.c (gen_producer_string): Ignore > OPT_fdiagnostics_nn_line_numbers. > * lto-wrapper.c (merge_and_complain): Handle > OPT_fdiagnostics_nn_line_numbers. > (append_compiler_options): Likewise. > (append_diag_options): Likewise. > * opts.c (common_handle_option): Likewise. > * toplev.c (general_init): Initialize > global_dc->use_nn_for_line_numbers_p. Reminds me a lot of the option to not print insn numbers and certain addresses in RTL dumps -- which makes comparing them easier.
OK jeff >
