[clang] [llvm] [clang][CoverageMapping] Refactor when setting MC/DC True/False (PR #78202)

NAKAMURA Takumi via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 16 22:57:39 PST 2024


================
@@ -1847,30 +1850,18 @@ struct CounterCoverageMappingBuilder
     // Extract the Parent Region Counter.
     Counter ParentCnt = getRegion().getCounter();
 
-    // Extract the MCDC condition IDs (returns 0 if not needed).
-    MCDCConditionID NextOrID = MCDCBuilder.getNextLOrCondID();
-    MCDCConditionID NextAndID = MCDCBuilder.getNextLAndCondID();
     MCDCConditionID LHSid = MCDCBuilder.getCondID(E->getLHS());
     MCDCConditionID RHSid = MCDCBuilder.getCondID(E->getRHS());
----------------
chapuni wrote:

They may be moved to `createBranchRegion()` since `CondID` is available as `CondIDMap`.

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


More information about the cfe-commits mailing list