================ @@ -234,8 +246,20 @@ struct MapRegionCounters : public RecursiveASTVisitor<MapRegionCounters> { if (Hash.getHashVersion() == PGO_HASH_V1) return Base::TraverseIfStmt(If); + // When single byte coverage mode is enabled, add a counter to then and + // else. + for (Stmt *CS : If->children()) { + if (!CS || !llvm::EnableSingleByteCoverage) ---------------- ZequanWu wrote:
Maybe hoist `llvm::EnableSingleByteCoverage` outside the loop as this is not going to change. Same for the following. https://github.com/llvm/llvm-project/pull/75425 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits