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

Jessica Paquette via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 5 20:49:37 PST 2025


================
@@ -494,10 +498,10 @@ struct MCDCRecord {
   /// TestVectors requires a translation from a ordinal position to actual
   /// condition ID. This is done via PosToID[].
   bool isConditionIndependencePairCovered(unsigned Condition) const {
+    assert(IndependencePairs);
     auto It = PosToID.find(Condition);
-    if (It != PosToID.end())
-      return IndependencePairs.contains(It->second);
-    llvm_unreachable("Condition ID without an Ordinal mapping");
+    assert(It != PosToID.end() && "Condition ID without an Ordinal mapping");
----------------
ornata wrote:

why is this changed to an assert from unreachable?

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


More information about the llvm-commits mailing list