mirrors/llvm-project
0
0
Fork 0
mirror of https://github.com/llvm/llvm-project.git synced 2025-04-29 11:56:07 +00:00
Code Issues Projects Releases Wiki Activity

Revert "[Coverage] Ignore unused functions if the count is 0." (#107901)

Browse Source 
Reverts llvm/llvm-project#107661

Breaks llvm-project/llvm/unittests/ProfileData/CoverageMappingTest.cpp
This commit is contained in:
Zequan Wu 2024-09-09 11:34:13 -07:00 committed by GitHub
parent 02fff933d0
commit a7c26aaf2e
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 1 additions and 56 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

55
compiler-rt/test/profile/instrprof-merging-2.cpp
View File

@ -1,55 +0,0 @@
// UNSUPPORTED: target={{.*windows.*}}
// clang-format off
// RUN: split-file %s %t
// RUN: %clangxx_profgen -fcoverage-mapping %t/test1.cpp -o %t/test1.exe
// RUN: %clangxx_profgen -fcoverage-mapping %t/test2.cpp -o %t/test2.exe
// RUN: env LLVM_PROFILE_FILE=%t/test1.profraw %run %t/test1.exe
// RUN: env LLVM_PROFILE_FILE=%t/test2.profraw %run %t/test2.exe
// RUN: llvm-profdata merge %t/test1.profraw %t/test2.profraw -o %t/merged.profdata
// RUN: llvm-cov show -instr-profile=%t/merged.profdata -object %t/test1.exe %t/test2.exe | FileCheck %s
// RUN: llvm-cov show -instr-profile=%t/merged.profdata -object %t/test2.exe %t/test1.exe | FileCheck %s
// CHECK: |struct Test {
// CHECK-NEXT: 1| int getToTest() {
// CHECK-NEXT: 2| for (int i = 0; i < 1; i++) {
// CHECK-NEXT: 1| if (false) {
// CHECK-NEXT: 0| return 1;
// CHECK-NEXT: 0| }
// CHECK-NEXT: 1| }
// CHECK-NEXT: 1| if (true) {
// CHECK-NEXT: 1| return 1;
// CHECK-NEXT: 1| }
// CHECK-NEXT: 0| return 1;
// CHECK-NEXT: 1| }
// CHECK-NEXT: |};
// CHECK-NEXT: |
#--- test.h
struct Test {
int getToTest() {
for (int i = 0; i < 1; i++) {
if (false) {
return 1;
}
}
if (true) {
return 1;
}
return 1;
}
};
#--- test1.cpp
#include "test.h"
int main() {
Test t;
t.getToTest();
return 0;
}
#--- test2.cpp
#include "test.h"
int main() {
return 0;
}

2
llvm/lib/ProfileData/Coverage/CoverageMapping.cpp
View File

@ -851,7 +851,7 @@ Error CoverageMapping::loadFunctionRecord(
// won't (in which case we don't unintuitively report functions as uncovered
// when they have non-zero counts in the profile).
if (Record.MappingRegions.size() == 1 &&
Record.MappingRegions[0].Count.isZero())
Record.MappingRegions[0].Count.isZero() && Counts[0] > 0)
return Error::success();
MCDCDecisionRecorder MCDCDecisions;