[clang] [Coverage] Introduce `getBranchCounterPair()`. NFC. (PR #112702)

NAKAMURA Takumi via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 7 23:06:46 PST 2025


================
@@ -938,6 +938,37 @@ struct CounterCoverageMappingBuilder
     return Counter::getCounter(CounterMap[S]);
   }
 
+  struct BranchCounterPair {
+    Counter Executed;
----------------
chapuni wrote:

I still wonder which pair of names would fit best. I adopted "Exec" from `BinOp` stuff and applied "Skipped" since "Exit" would be an exaggeration. "Checked/Skipped" would make less sense. OTOH, "Taken/NotTaken" looks redundant. Better idea?

I want them self-explained.

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


More information about the cfe-commits mailing list