From aaa49273cfa4857a7c1d35334e7b46c51edacaf8 Mon Sep 17 00:00:00 2001 From: Geza Lore Date: Wed, 1 Oct 2025 21:20:50 +0100 Subject: [PATCH] Internals: Fix coverage exclusion markers --- include/verilated_sc_trace.h | 2 +- src/V3Ast.cpp | 2 +- src/V3Combine.cpp | 2 +- src/V3Const.cpp | 2 +- src/V3Delayed.cpp | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/include/verilated_sc_trace.h b/include/verilated_sc_trace.h index 4a8e18f30..1f17fda2c 100644 --- a/include/verilated_sc_trace.h +++ b/include/verilated_sc_trace.h @@ -180,7 +180,7 @@ private: // Cadence Incisive has these as abstract functions so we must create them void set_time_unit(int exponent10_seconds) override {} // deprecated #endif - void set_time_unit(double v, sc_core::sc_time_unit tu) override {} // LCOV_EXCL_LINE + void set_time_unit(double v, sc_core::sc_time_unit tu) override {} //-------------------------------------------------- // SystemC 2.1.v1 diff --git a/src/V3Ast.cpp b/src/V3Ast.cpp index deb546937..a51828d20 100644 --- a/src/V3Ast.cpp +++ b/src/V3Ast.cpp @@ -1281,7 +1281,7 @@ char* AstNode::dumpTreeJsonGdb(intptr_t nodep) { return dumpTreeJsonGdb(reinterpret_cast(nodep)); } // cppcheck-suppress unusedFunction // Debug only -void AstNode::dumpGdb(const AstNode* nodep) { // For GDB only // LCOV_EXCL_LINE +void AstNode::dumpGdb(const AstNode* nodep) { // For GDB only // LCOV_EXCL_START if (!nodep) { cout << "" << endl; return; diff --git a/src/V3Combine.cpp b/src/V3Combine.cpp index 8c7bce596..767dc84d7 100644 --- a/src/V3Combine.cpp +++ b/src/V3Combine.cpp @@ -214,7 +214,7 @@ class CombineVisitor final : VNVisitor { // only used in tracing functions, which are not combined. Blow up in case this changes. nodep->v3fatalSrc( "Don't know how to combine functions that are referenced via AstAddrOfCFunc"); - // LCOV_EXCL_END + // LCOV_EXCL_STOP } //-------------------- diff --git a/src/V3Const.cpp b/src/V3Const.cpp index 7655282f6..75c5c632a 100644 --- a/src/V3Const.cpp +++ b/src/V3Const.cpp @@ -805,7 +805,7 @@ public: cout << "- Needs flipping: " << needsFlip << "\n"; cout << "- Needs cleaning: " << needsCleaning << "\n"; cout << "- Size: " << resultOps << " input size: " << visitor.m_ops << "\n"; - } // LCOV_EXCL_END + } // LCOV_EXCL_STOP // Sometimes we have no terms left after ignoring redundant terms // (all of which were zeroes) diff --git a/src/V3Delayed.cpp b/src/V3Delayed.cpp index 6e1a704bf..5d774a42d 100644 --- a/src/V3Delayed.cpp +++ b/src/V3Delayed.cpp @@ -1007,7 +1007,7 @@ class DelayedVisitor final : public VNVisitor { switch (vscpInfo.m_scheme) { case Scheme::Undecided: // LCOV_EXCL_START UASSERT_OBJ(false, vscp, "Failed to choose NBA scheme"); - break; + break; // LCOV_EXCL_STOP case Scheme::UnsupportedCompoundArrayInLoop: { // Will report error at the site of the NBA break;