From: Clifford Wolf <clifford@clifford.at>
Date: Mon, 29 Jun 2015 23:38:34 +0000 (+0200)
Subject: Bugfix in chparam
X-Git-Tag: yosys-0.6~239
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=77e89399a659e5c16e5cd6a8040e38e811e7192d;p=yosys.git

Bugfix in chparam
---

diff --git a/passes/cmds/setattr.cc b/passes/cmds/setattr.cc
index a82625696..58fcf028f 100644
--- a/passes/cmds/setattr.cc
+++ b/passes/cmds/setattr.cc
@@ -45,7 +45,7 @@ struct setunset_t
 	}
 };
 
-static void do_setunset(dict<RTLIL::IdString, RTLIL::Const> &attrs, std::vector<setunset_t> &list)
+static void do_setunset(dict<RTLIL::IdString, RTLIL::Const> &attrs, const std::vector<setunset_t> &list)
 {
 	for (auto &item : list)
 		if (item.unset)
@@ -217,6 +217,8 @@ struct ChparamPass : public Pass {
 		}
 		extra_args(args, argidx, design);
 
+		do_setunset(new_parameters, setunset_list);
+
 		if (list_mode) {
 			if (!new_parameters.empty())
 				log_cmd_error("The options -set and -list cannot be used together.\n");
@@ -229,11 +231,8 @@ struct ChparamPass : public Pass {
 		}
 
 		pool<IdString> modnames, old_modnames;
-		for (auto module : design->selected_modules()) {
-			if (design->selected_whole_module(module))
-				modnames.insert(module->name);
-			else
-				log_warning("Ignoring partially selected module %s.\n", log_id(module));
+		for (auto module : design->selected_whole_modules_warn()) {
+			modnames.insert(module->name);
 			old_modnames.insert(module->name);
 		}
 		modnames.sort();