[clang] [compiler-rt] [llvm] [InstrProf] Single byte counters in coverage (PR #75425)

NAKAMURA Takumi via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 27 14:18:46 PST 2024


================
@@ -1503,38 +1527,53 @@ struct CounterCoverageMappingBuilder
     }
 
     // Create Branch Region around condition.
-    createBranchRegion(S->getCond(), BodyCount,
-                       subtractCounters(CondCount, BodyCount));
+    if (!llvm::EnableSingleByteCoverage)
+      createBranchRegion(S->getCond(), BodyCount,
+                         subtractCounters(CondCount, BodyCount));
----------------
chapuni wrote:

As far as I have checked, I guess `!EnableSingleByteCoverage` can be sunk into `createBranchRegion()` rather than checking from callers.

Sorry for my delayed comment. I didn't notice till merging.

https://github.com/llvm/llvm-project/pull/75425


More information about the cfe-commits mailing list