[PATCH 06/17] scripts/code_cov_parse_info: prepare $all_branch to get function data
Kamil Konieczny
kamil.konieczny at linux.intel.com
Thu Feb 15 16:41:13 UTC 2024
Hi Mauro,
On 2024-02-15 at 11:27:15 +0100, Mauro Carvalho Chehab wrote:
> From: Mauro Carvalho Chehab <mchehab at kernel.org>
>
> Right now, we store only the branch counts. We also need to store
> the functions that each branch belongs. So, add a new hash in
> order to allow adding function data later on.
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab at kernel.org>
Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> ---
> scripts/code_cov_parse_info | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/scripts/code_cov_parse_info b/scripts/code_cov_parse_info
> index 1be7fddf3f3e..d28217cc196d 100755
> --- a/scripts/code_cov_parse_info
> +++ b/scripts/code_cov_parse_info
> @@ -233,7 +233,7 @@ sub parse_info_data($)
> $was_used = 1 if ($taken > 0);
>
> $record{$source}{$func}{brda}{$where} += $taken;
> - $all_branch{$source}{"$where"} += $taken;
> + $all_branch{$source}{"$where"}{count} += $taken;
> next;
> }
>
> @@ -448,7 +448,7 @@ sub gen_stats()
>
> foreach my $where (keys(%{$all_branch{$source}})) {
> $stats{"branch_count"}++;
> - $stats{"branch_reached"}++ if ($all_branch{$source}{$where} != 0);
> + $stats{"branch_reached"}++ if ($all_branch{$source}{$where}{count} != 0);
> }
> }
>
> @@ -607,7 +607,7 @@ sub generate_report($)
> }
> foreach my $source (keys(%{$report{$f}{"all_branch"}})) {
> foreach my $where (keys(%{$report{$f}{"all_branch"}{$source}})) {
> - $all_branch{$source}{"$where"} += $report{$f}{"all_branch"}{$source}{$where};
> + $all_branch{$source}{"$where"}{count} += $report{$f}{"all_branch"}{$source}{$where}{count};
> }
> }
> for my $source(keys(%{$report{$f}{"files"}})) {
> @@ -887,7 +887,7 @@ sub check_source_branches()
>
> my @lines;
> foreach my $where (sort keys %{$all_branch{$source}}) {
> - my $taken = $all_branch{$source}{$where};
> + my $taken = $all_branch{$source}{$where}{count};
> next if ($taken > 0);
>
> next if !($where =~ m/^(-?\d+),(-?\d+),(-?\d+)/);
> --
> 2.43.0
>
More information about the igt-dev
mailing list