From 3cb8d6ab0313f271117c9a51e524f16531b36593 Mon Sep 17 00:00:00 2001 From: Alain Dargelas Date: Mon, 4 Nov 2024 18:11:09 -0800 Subject: [PATCH] review and warnings --- passes/cmds/activity.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/passes/cmds/activity.cc b/passes/cmds/activity.cc index 592dffa89..ab85a387c 100644 --- a/passes/cmds/activity.cc +++ b/passes/cmds/activity.cc @@ -76,13 +76,13 @@ struct ActivityProp { std::vector activities = tokenize(act, " ", true); std::vector duties = tokenize(duty, " ", true); // Assign them to each SigBit (1 signal bit) - for (int i = 0; i < GetSize(sig); i++) { + for (uint32_t i = 0; i < (uint32_t)GetSize(sig); i++) { SigBit bit(sig[i]); - if (i <= activities.size() -1) { + if (i < activities.size()) { ActivityMap.emplace(bit, activities[i]); DutyMap.emplace(bit, duties[i]); } else { - log_warning("Zeroing out activity for module: %s, wire: %s, wiresize: %d, actisize: %d", module->name.c_str(), wire->name.c_str(), GetSize(sig), activities.size()); + log_warning("Zeroing out activity for module: %s, wire: %s, wiresize: %d, actisize: %ld", module->name.c_str(), wire->name.c_str(), GetSize(sig), activities.size()); ActivityMap.emplace(bit, "0.0"); DutyMap.emplace(bit, "0.0"); }