From 7ba381a0d0d8465602f5e78771bab347cec5bd0a Mon Sep 17 00:00:00 2001
From: Martin Whitaker <icarus@martin-whitaker.me.uk>
Date: Thu, 4 Nov 2021 19:35:22 +0000
Subject: [PATCH] Remove buggy assertion.

If this assert fires, the "this" pointer we pass to it will be a
null pointer, so will cause a null pointer dereference. We've
tested it is not null earlier, so we don't need the assertion.
---
 net_design.cc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/net_design.cc b/net_design.cc
index 44e7079e4..5ab26a15a 100644
--- a/net_design.cc
+++ b/net_design.cc
@@ -613,7 +613,6 @@ void NetScope::evaluate_parameter_logic_(Design*des, param_ref_t cur)
       }
 
       NetEConst*val = dynamic_cast<NetEConst*>((*cur).second.val);
-      ivl_assert(*(*cur).second.val, (*cur).second.val);
       ivl_assert(*(*cur).second.val, val);
 
       verinum value = val->value();