From 77e89399a659e5c16e5cd6a8040e38e811e7192d Mon Sep 17 00:00:00 2001
From: Clifford Wolf <clifford@clifford.at>
Date: Tue, 30 Jun 2015 01:38:34 +0200
Subject: [PATCH] Bugfix in chparam

---
 passes/cmds/setattr.cc | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

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();
-- 
2.30.2