From 769c0fa318ccd42c176f054c2966d6c0b2566ae9 Mon Sep 17 00:00:00 2001 From: Paul Wright <68547250+polmacanceart@users.noreply.github.com> Date: Sun, 10 Jan 2021 17:40:03 +0000 Subject: [PATCH] Fix to ignore coverage on real ports (#2741) (#2745) --- docs/CONTRIBUTORS | 1 + src/V3AstNodes.h | 2 +- test_regress/t/t_cover_toggle.v | 4 +++- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/docs/CONTRIBUTORS b/docs/CONTRIBUTORS index 363676c91..67b634c82 100644 --- a/docs/CONTRIBUTORS +++ b/docs/CONTRIBUTORS @@ -53,6 +53,7 @@ Nandu Raj Nathan Kohagen Nathan Myers Patrick Stewart +Paul Wright Peter Horvath Peter Monsson Philipp Wagner diff --git a/src/V3AstNodes.h b/src/V3AstNodes.h index 117d28b50..b6bb6653a 100644 --- a/src/V3AstNodes.h +++ b/src/V3AstNodes.h @@ -2170,7 +2170,7 @@ public: return ((isIO() || isSignal()) && (isIO() || isBitLogic()) // Wrapper would otherwise duplicate wrapped module's coverage - && !isSc() && !isPrimaryIO() && !isConst()); + && !isSc() && !isPrimaryIO() && !isConst() && !isDouble()); } bool isClassMember() const { return varType() == AstVarType::MEMBER; } bool isStatementTemp() const { return (varType() == AstVarType::STMTTEMP); } diff --git a/test_regress/t/t_cover_toggle.v b/test_regress/t/t_cover_toggle.v index e069a3382..1c4008c3b 100644 --- a/test_regress/t/t_cover_toggle.v +++ b/test_regress/t/t_cover_toggle.v @@ -6,10 +6,12 @@ module t (/*AUTOARG*/ // Inputs - clk + clk, + check_real ); input clk; + input real check_real; // Check issue #2741 typedef struct packed { union packed {