[llvm] [Coverage] MCDC: Move findIndependencePairs deferred into MCDCRecord (PR #121188)

NAKAMURA Takumi via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 5 23:04:15 PST 2025


================
@@ -507,7 +511,8 @@ struct MCDCRecord {
   /// via PosToID[].
   TVRowPair getConditionIndependencePair(unsigned Condition) {
     assert(isConditionIndependencePairCovered(Condition));
-    return IndependencePairs[PosToID[Condition]];
+    assert(IndependencePairs);
----------------
chapuni wrote:

It's redundant since `isConditionIndependencePairCovered` checks `IndependencePairs`. That said, I want to reconfirm before its use `*IndependencePairs`.

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


More information about the llvm-commits mailing list