Bugfix in hierarchy handling of blackbox module ports

Signed-off-by: Clifford Wolf <clifford@clifford.at>

frontends/ast/ast.cc

@@ -1067,7 +1067,7 @@ AstModule::~AstModule()
 }
 
 // create a new parametric module (when needed) and return the name of the generated module
-RTLIL::IdString AstModule::derive(RTLIL::Design *design, dict<RTLIL::IdString, RTLIL::Const> parameters)
+RTLIL::IdString AstModule::derive(RTLIL::Design *design, dict<RTLIL::IdString, RTLIL::Const> parameters, bool)
 {
 	std::string stripped_name = name.str();
 

frontends/ast/ast.h

@@ -283,7 +283,7 @@ namespace AST
 		AstNode *ast;
 		bool nolatches, nomeminit, nomem2reg, mem2reg, lib, noopt, icells, autowire;
 		virtual ~AstModule();
-		virtual RTLIL::IdString derive(RTLIL::Design *design, dict<RTLIL::IdString, RTLIL::Const> parameters);
+		virtual RTLIL::IdString derive(RTLIL::Design *design, dict<RTLIL::IdString, RTLIL::Const> parameters, bool mayfail);
 		virtual RTLIL::Module *clone() const;
 	};
 

kernel/rtlil.cc

@@ -639,8 +639,10 @@ RTLIL::Module::~Module()
 		delete it->second;
 }
 
-RTLIL::IdString RTLIL::Module::derive(RTLIL::Design*, dict<RTLIL::IdString, RTLIL::Const>)
+RTLIL::IdString RTLIL::Module::derive(RTLIL::Design*, dict<RTLIL::IdString, RTLIL::Const>, bool mayfail)
 {
+	if (mayfail)
+		return RTLIL::IdString();
 	log_error("Module `%s' is used with parameters but is not parametric!\n", id2cstr(name));
 }
 

kernel/rtlil.h

@@ -906,7 +906,7 @@ public:
 
 	Module();
 	virtual ~Module();
-	virtual RTLIL::IdString derive(RTLIL::Design *design, dict<RTLIL::IdString, RTLIL::Const> parameters);
+	virtual RTLIL::IdString derive(RTLIL::Design *design, dict<RTLIL::IdString, RTLIL::Const> parameters, bool mayfail = false);
 	virtual size_t count_id(RTLIL::IdString id);
 
 	virtual void sort();

passes/hierarchy/hierarchy.cc

@@ -625,16 +625,15 @@ struct HierarchyPass : public Pass {
 		for (auto module : design->modules())
 		for (auto cell : module->cells())
 		{
-			if (GetSize(cell->parameters) != 0)
-				continue;
-
 			Module *m = design->module(cell->type);
 
 			if (m == nullptr)
 				continue;
 
-			if (m->get_bool_attribute("\\blackbox") && cell->parameters.size()) {
+			if (m->get_bool_attribute("\\blackbox") && !cell->parameters.empty()) {
-				IdString new_m_name = m->derive(design, cell->parameters);
+				IdString new_m_name = m->derive(design, cell->parameters, true);
+				if (new_m_name.empty())
+					continue;
 				if (new_m_name != m->name) {
 					m = design->module(new_m_name);
 					blackbox_derivatives.insert(m);