From bd2ac68769ebddf8c239eeaf26d43d30453db583 Mon Sep 17 00:00:00 2001 From: Andrew Zonenberg Date: Mon, 14 Aug 2017 11:16:44 -0700 Subject: [PATCH] rmports now works on all modules in the design, not just the top. --- passes/opt/rmports.cc | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/passes/opt/rmports.cc b/passes/opt/rmports.cc index 30cf149ce..f08d7b126 100644 --- a/passes/opt/rmports.cc +++ b/passes/opt/rmports.cc @@ -27,7 +27,7 @@ USING_YOSYS_NAMESPACE PRIVATE_NAMESPACE_BEGIN struct RmportsPassPass : public Pass { - RmportsPassPass() : Pass("rmports", "remove top-level ports with no connections") { } + RmportsPassPass() : Pass("rmports", "remove module ports with no connections") { } virtual void help() { // |---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---| @@ -42,7 +42,10 @@ struct RmportsPassPass : public Pass { virtual void execute(std::vector /*args*/, RTLIL::Design *design) { log_header(design, "Executing RMPORTS pass (remove top level ports with no connections).\n"); - ProcessModule(design->top_module()); + + auto modules = design->selected_modules(); + for(auto mod : modules) + ProcessModule(mod); } virtual void ProcessModule(RTLIL::Module* module) @@ -114,7 +117,7 @@ struct RmportsPassPass : public Pass { //Print the ports out as we go through them for(auto port : unused_ports) { - log(" removing unused top-level port %s\n", port.c_str()); + log(" removing unused port %s\n", port.c_str()); //Remove from ports list for(size_t i=0; iports.size(); i++) @@ -132,7 +135,7 @@ struct RmportsPassPass : public Pass { wire->port_output = false; wire->port_id = 0; } - log("Removed %zu unused top-level ports.\n", unused_ports.size()); + log("Removed %zu unused ports.\n", unused_ports.size()); //Re-number all of the wires that DO have ports still on them for(size_t i=0; iports.size(); i++) -- 2.30.2