[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