[llvm-branch-commits] [compiler-rt] [llvm] [Coverage] Make SingleByteCoverage work consistent to merging (PR #110972)

Ellis Hoag via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Thu Oct 3 11:47:08 PDT 2024


================
@@ -952,7 +952,7 @@ void InstrProfRecord::merge(InstrProfRecord &Other, uint64_t Weight,
       Value = getInstrMaxCountValue();
       Overflowed = true;
     }
-    Counts[I] = Value;
----------------
ellishg wrote:

This is deliberate. Even though we only record boolean coverage in the raw profiles, when we aggregate many raw profiles together we can still get some sense of relative hotness by looking at the counter value. Otherwise we lose information if we treat the counter value in the indexed profile as a boolean.

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


More information about the llvm-branch-commits mailing list