[compiler-rt] [llvm] Revert "[Coverage] Ignore unused functions if the count is 0." (PR #107901)

[Zequan Wu via llvm-commits]

https://github.com/ZequanWu created https://github.com/llvm/llvm-project/pull/107901

Reverts llvm/llvm-project#107661

Breaks llvm-project/llvm/unittests/ProfileData/CoverageMappingTest.cpp

>From 1b8479684c74caa82e0b5192a780b75f1000a9c2 Mon Sep 17 00:00:00 2001
From: Zequan Wu <zequanwu at google.com>
Date: Mon, 9 Sep 2024 14:33:27 -0400
Subject: [PATCH] Revert "[Coverage] Ignore unused functions if the count is 0.
 (#107661)"

This reverts commit 6850410562123b6e4fbb039e7ba4a2325b994b84.
---
 .../test/profile/instrprof-merging-2.cpp      | 55 -------------------
 .../ProfileData/Coverage/CoverageMapping.cpp  |  2 +-
 2 files changed, 1 insertion(+), 56 deletions(-)
 delete mode 100644 compiler-rt/test/profile/instrprof-merging-2.cpp

diff --git a/compiler-rt/test/profile/instrprof-merging-2.cpp b/compiler-rt/test/profile/instrprof-merging-2.cpp
deleted file mode 100644
index 438394f9fb239e..00000000000000
--- a/compiler-rt/test/profile/instrprof-merging-2.cpp
+++ /dev/null
@@ -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;
-}
diff --git a/llvm/lib/ProfileData/Coverage/CoverageMapping.cpp b/llvm/lib/ProfileData/Coverage/CoverageMapping.cpp
index acb7dd922ab9fa..18643c6b44485e 100644
--- a/llvm/lib/ProfileData/Coverage/CoverageMapping.cpp
+++ b/llvm/lib/ProfileData/Coverage/CoverageMapping.cpp
@@ -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;