Skip to content

Commit

Permalink
[llvm-cov[ Fix lcov coverage report contains functions from other com…
Browse files Browse the repository at this point in the history
…pilation units.

Summary: Patch by Chuan Qiu (@eagleonhill).

Reviewers: Dor1s

Reviewed By: Dor1s

Subscribers: lebedev.ri, llvm-commits

Tags: #llvm

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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@364653 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
Dor1s committed Jun 28, 2019
1 parent 4f9b5ab commit cdc9397
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 1 deletion.
12 changes: 12 additions & 0 deletions test/tools/llvm-cov/multiple-files.test
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
// RUN: llvm-profdata merge %S/Inputs/multiple-files.proftext -o %t.profdata
// RUN: llvm-cov report %S/Inputs/multiple-files.covmapping -instr-profile %t.profdata | FileCheck %s -check-prefix=MANY_COMPONENTS
// RUN: llvm-cov report %S/Inputs/multiple-files2.covmapping -instr-profile %t.profdata | FileCheck %s -check-prefix=ONE_COMPONENT
// RUN: llvm-cov export %S/Inputs/multiple-files.covmapping -instr-profile %t.profdata -format=lcov | FileCheck %s -check-prefix=LCOV

// MANY_COMPONENTS: Filename
// MANY_COMPONENTS-NEXT: ---
Expand All @@ -13,3 +14,14 @@
// ONE_COMPONENT-NEXT: ---
// ONE_COMPONENT-NEXT: {{^}}cov.c
// ONE_COMPONENT-NEXT: {{^}}cov.h

// LCOV-LABEL: SF:{{.*}}a{{[/\\]}}f2.c
// LCOV: FN:1,f2
// No extra funcs
// LCOV-NOT: FN:
// LCOV-LABEL: SF:{{.*}}b{{[/\\]}}c{{[/\\]}}f4.c
// LCOV: FN:1,f4
// LCOV-LABEL: SF:{{.*}}b{{[/\\]}}f3.c
// LCOV: FN:1,f3
// LCOV-LABEL: SF:{{.*}}f1.c
// LCOV: FN:1,f1
2 changes: 1 addition & 1 deletion tools/llvm-cov/CoverageExporterLcov.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ void renderFile(raw_ostream &OS, const coverage::CoverageMapping &Coverage,
OS << "SF:" << Filename << '\n';

if (!ExportSummaryOnly) {
renderFunctions(OS, Coverage.getCoveredFunctions());
renderFunctions(OS, Coverage.getCoveredFunctions(Filename));
}
renderFunctionSummary(OS, FileReport);

Expand Down

0 comments on commit cdc9397

Please sign in to comment.