From e59b5e24ea78b6ca06da7ba07ec3218d6b65e989 Mon Sep 17 00:00:00 2001 From: Martin Liska Date: Tue, 27 Feb 2018 15:11:08 +0100 Subject: [PATCH] Make groups more generic (PR gcov-profile/84548). 2018-02-27 Martin Liska PR gcov-profile/84548 * gcov.c (process_file): Allow partial overlap and consider it also as group functions. (output_lines): Properly calculate range of lines for a group. 2018-02-27 Martin Liska PR gcov-profile/84548 * g++.dg/gcov/pr84548.C: New test. From-SVN: r258033 --- gcc/ChangeLog | 7 +++++++ gcc/gcov.c | 10 ++++++++-- gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/g++.dg/gcov/pr84548.C | 19 +++++++++++++++++++ 4 files changed, 39 insertions(+), 2 deletions(-) create mode 100644 gcc/testsuite/g++.dg/gcov/pr84548.C diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 1a8edf3c344..a230b12df57 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2018-02-27 Martin Liska + + PR gcov-profile/84548 + * gcov.c (process_file): Allow partial overlap and consider it + also as group functions. + (output_lines): Properly calculate range of lines for a group. + 2018-02-27 Martin Liska * timevar.c (timer::print_row): Remove 'usr', 'sys', 'wall' and diff --git a/gcc/gcov.c b/gcc/gcov.c index 37f431c0e91..6bbfe33ca33 100644 --- a/gcc/gcov.c +++ b/gcc/gcov.c @@ -1151,7 +1151,6 @@ process_file (const char *file_name) function_info **slot = fn_map.get (needle); if (slot) { - gcc_assert ((*slot)->end_line == (*it)->end_line); (*slot)->is_group = 1; (*it)->is_group = 1; } @@ -2957,7 +2956,14 @@ output_lines (FILE *gcov_file, const source_info *src) { fns = src->get_functions_at_location (line_num); if (fns.size () > 1) - line_start_group = fns[0]->end_line; + { + /* It's possible to have functions that partially overlap, + thus take the maximum end_line of functions starting + at LINE_NUM. */ + for (unsigned i = 0; i < fns.size (); i++) + if (fns[i]->end_line > line_start_group) + line_start_group = fns[i]->end_line; + } else if (fns.size () == 1) { function_info *fn = fns[0]; diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 84ac3ee6046..6506d8f4c6b 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2018-02-27 Martin Liska + + PR gcov-profile/84548 + * g++.dg/gcov/pr84548.C: New test. + 2018-02-27 Jakub Jelinek PR target/84575 diff --git a/gcc/testsuite/g++.dg/gcov/pr84548.C b/gcc/testsuite/g++.dg/gcov/pr84548.C new file mode 100644 index 00000000000..6c22c1902f2 --- /dev/null +++ b/gcc/testsuite/g++.dg/gcov/pr84548.C @@ -0,0 +1,19 @@ +// PR gcov-profile/84548 +// { dg-options "-fprofile-arcs -ftest-coverage" } +// { dg-do run { target native } } +// TODO: add support for groups to gcov.exp script + +struct A { static int foo () { return 1; }; static int bar () { + int x; + return 2; } }; + +int main() +{ + int a = A::foo () + A::bar (); + if (a != 3) + return 1; + + return 0; +} + +// { dg-final { run-gcov remove-gcda pr84548.C } } -- 2.30.2