================ @@ -939,8 +939,17 @@ struct CounterCoverageMappingBuilder Counter Skipped; }; - BranchCounterPair getBranchCounterPair(const Stmt *S, Counter ParentCnt) { + BranchCounterPair + getBranchCounterPair(const Stmt *S, Counter ParentCnt, + std::optional<Counter> SkipCntForOld = std::nullopt) { Counter ExecCnt = getRegionCounter(S); + + // The old behavior of SingleByte shouldn't emit Branches. + if (llvm::EnableSingleByteCoverage) { + assert(SkipCntForOld); ---------------- chapuni wrote:
I wish its end of life would come soon. The assertion will disappear when implementation begins. e.g. https://github.com/llvm/llvm-project/pull/113111/commits/3c6252260ee11e3a453076b4d96ffffe20d49998 I really want to label it `&& "UNDER CONSTRUCTION"`, or eliminate this assertions now. https://github.com/llvm/llvm-project/pull/112702 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits