diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 05c2b2984be..6f8ed5a2a85 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2007-08-16 Seongbae Park + + * tree-eh.c (lower_try_finally_onedest): Reset the locus + of GOTO that's relocated to a different block. + 2007-08-16 Kaveh R. Ghazi * c-common.c (const_strip_array_types): New. diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 387b5791921..37532f52131 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2007-08-16 Seongbae Park + + * g++.dg/gcov/gcov-4.C: New test. + * lib/gcov.exp (verify-lines): Add support for + checking no-coverage lines. + 2007-08-16 Victor Kaplansky * gcc.dg/vect/vect-vfa-01.c: New. diff --git a/gcc/testsuite/g++.dg/gcov/gcov-4.C b/gcc/testsuite/g++.dg/gcov/gcov-4.C new file mode 100644 index 00000000000..2f83ff1becc --- /dev/null +++ b/gcc/testsuite/g++.dg/gcov/gcov-4.C @@ -0,0 +1,41 @@ +/* Check that execution counts and branch probabilities for various C + constructs are reported correctly by gcov. */ + +#include + +/* { dg-options "-fprofile-arcs -ftest-coverage -fno-exceptions" } */ +/* { dg-do run { target native } } */ + +class foo { +public: + foo() { printf("foo()\n"); }; + ~foo() { printf("~foo()\n"); }; + void method(void) { printf("method()\n"); }; +}; + +int func(int i, int j) __attribute__((noinline)); + +int func(int i, int j){ + if (j) { + printf("unreachable\n"); + return 3; + } + + foo f; + + if (i == 1) { + f.method(); + f.method(); + } else { + f.method(); + printf("unreachable\n"); + return 2; /* count(#####) */ + } + f.method(); + return 0; +} + +int main() { + return func(1, 0); +} +/* { dg-final { run-gcov gcov-4.C } } */ diff --git a/gcc/testsuite/lib/gcov.exp b/gcc/testsuite/lib/gcov.exp index 35e86958df9..1c7484785db 100644 --- a/gcc/testsuite/lib/gcov.exp +++ b/gcc/testsuite/lib/gcov.exp @@ -42,7 +42,9 @@ proc verify-lines { testcase file } { set failed 0 set fd [open $file r] while { [gets $fd line] >= 0 } { - if [regexp "^ *(\[^:]*): *(\[0-9\]+):.*count\\((\[0-9\]+)\\)" \ + # We want to match both "-" and "#####" as count as well as numbers, + # since we want to detect lines that shouldn't be marked as covered. + if [regexp "^ *(\[^:]*): *(\[0-9\\-#]+):.*count\\((\[0-9\\-#]+)\\)" \ "$line" all is n shouldbe] { if { $is == "" } { fail "$n:no data available for this line" diff --git a/gcc/tree-eh.c b/gcc/tree-eh.c index 659bbfdad25..89f863fb85f 100644 --- a/gcc/tree-eh.c +++ b/gcc/tree-eh.c @@ -1020,6 +1020,9 @@ lower_try_finally_onedest (struct leh_state *state, struct leh_tf_state *tf) } } + /* Reset the locus of the goto since we're moving + goto to a different block which might be on a different line. */ + SET_EXPR_LOCUS (tf->goto_queue[0].cont_stmt, NULL); append_to_statement_list (tf->goto_queue[0].cont_stmt, tf->top_p); maybe_record_in_goto_queue (state, tf->goto_queue[0].cont_stmt); }