[clang] [llvm] [MC/DC][Coverage] Loosen the limit of NumConds from 6 (PR #82448)

Jessica Paquette via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 17 21:35:07 PDT 2024


================
@@ -190,18 +190,30 @@ class SourceMappingRegion {
 
   bool isBranch() const { return FalseCount.has_value(); }
 
+  bool isMCDCBranch() const {
+    const auto *BranchParams = std::get_if<mcdc::BranchParameters>(&MCDCParams);
+    assert(BranchParams == nullptr || BranchParams->ID >= 0);
----------------
ornata wrote:

I think it would be nice if this assert was not necessary. Is it possible to enforce that BranchParams->ID is never when we have MC/DC parameters?

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


More information about the cfe-commits mailing list