]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
Ensure function.end_line in source_info.lines
authorJørgen Kvalsvik <j@lambda.is>
Wed, 10 Jul 2024 16:47:27 +0000 (18:47 +0200)
committerJørgen Kvalsvik <j@lambda.is>
Thu, 11 Jul 2024 07:11:55 +0000 (09:11 +0200)
Ensure that the function.end_line in the lines vector for the source
file, even if it is not explicitly touched by a basic block. This
ensures consistency with what you would expect. For example, this file
has sources[sum.cc].lines.size () == 23 and main.end_line == 2 without
adjusting sources.lines, which in this case is a no-op.

    #####:   17:int main ()
        -:   18:{
    #####:   19:  sum (1, 2);
    #####:   20:  sum (1.1, 2);
    #####:   21:  sum (2.2, 2.3);
    #####:   22:}

This is a useful property when combined with selective reporting.

gcc/ChangeLog:

* gcov.cc (process_all_functions): Ensure fn.end_line is
included source[fn].lines.

gcc/gcov.cc

index 2e4bd9d3c5dab231016af8200296c8b534e02196..7b4a075c5db60e0b4bd0b9c674decd608f6db132 100644 (file)
@@ -1544,6 +1544,12 @@ process_all_functions (void)
                }
            }
 
+         /* Make sure to include the last line for this function even when it
+            is not directly covered by a basic block, for example when } is on
+            its own line.  */
+         if (sources[fn->src].lines.size () <= fn->end_line)
+           sources[fn->src].lines.resize (fn->end_line + 1);
+
          /* Allocate lines for group function, following start_line
             and end_line information of the function.  */
          if (fn->is_group)