[PATCH] D53110: [Coverage] Apply filtered paths to summary

[Chris Bieneman via Phabricator via llvm-commits]

beanz created this revision.
beanz added a reviewer: vsk.

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.


Repository:
  rL LLVM

https://reviews.llvm.org/D53110

Files:
  utils/prepare-code-coverage-artifact.py


Index: utils/prepare-code-coverage-artifact.py
===================================================================
--- utils/prepare-code-coverage-artifact.py
+++ utils/prepare-code-coverage-artifact.py
@@ -51,7 +51,8 @@
     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,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53110.169098.patch
Type: text/x-patch
Size: 707 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181010/bed3d645/attachment.bin>