Skip to content

Commit

Permalink
[Coverage] Apply filtered paths to summary
Browse files Browse the repository at this point in the history
Summary:
The script to generate code coverage reports supports passing filter paths to llvm-cov when generating the HTML reports, but doesn't pass those paths to the summary generation as well. This results in a summary report that doesn't match the HTML report.

This patch addresses the problem by also passing the filter paths to the summary report generation.

Reviewers: vsk

Subscribers: llvm-commits

Differential Revision: https://reviews.llvm.org/D53110

llvm-svn: 344217
  • Loading branch information
llvm-beanz committed Oct 11, 2018
1 parent 1bb9c85 commit 9a10eac
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion llvm/utils/prepare-code-coverage-artifact.py
Expand Up @@ -51,7 +51,8 @@ def prepare_html_report(host_llvm_cov, profile, report_dir, binaries,
subprocess.check_call(invocation)
with open(os.path.join(report_dir, 'summary.txt'), 'wb') as Summary:
subprocess.check_call([host_llvm_cov, 'report'] + objects +
['-instr-profile', profile], stdout=Summary)
['-instr-profile', profile] + restricted_dirs,
stdout=Summary)
print('Done!')

def prepare_html_reports(host_llvm_cov, profdata_path, report_dir, binaries,
Expand Down

0 comments on commit 9a10eac

Please sign in to comment.