From 05f24adca90828a453c70a0d729459f460b2fd70 Mon Sep 17 00:00:00 2001 From: Patrick Urban Date: Wed, 20 Oct 2021 09:07:01 +0200 Subject: [PATCH] synth_gatemate: Remove gatemate_bramopt pass --- techlibs/gatemate/Makefile.inc | 1 - techlibs/gatemate/gatemate_bramopt.cc | 146 -------------------------- techlibs/gatemate/synth_gatemate.cc | 1 - 3 files changed, 148 deletions(-) delete mode 100644 techlibs/gatemate/gatemate_bramopt.cc diff --git a/techlibs/gatemate/Makefile.inc b/techlibs/gatemate/Makefile.inc index bd18ce857..622febf48 100644 --- a/techlibs/gatemate/Makefile.inc +++ b/techlibs/gatemate/Makefile.inc @@ -1,6 +1,5 @@ OBJS += techlibs/gatemate/synth_gatemate.o -OBJS += techlibs/gatemate/gatemate_bramopt.o $(eval $(call add_share_file,share/gatemate,techlibs/gatemate/iob_map.v)) $(eval $(call add_share_file,share/gatemate,techlibs/gatemate/reg_map.v)) diff --git a/techlibs/gatemate/gatemate_bramopt.cc b/techlibs/gatemate/gatemate_bramopt.cc deleted file mode 100644 index bd16ba4c9..000000000 --- a/techlibs/gatemate/gatemate_bramopt.cc +++ /dev/null @@ -1,146 +0,0 @@ -/* - * yosys -- Yosys Open SYnthesis Suite - * - * Copyright (C) 2021 Cologne Chip AG - * - * Permission to use, copy, modify, and/or distribute this software for any - * purpose with or without fee is hereby granted, provided that the above - * copyright notice and this permission notice appear in all copies. - * - * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES - * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF - * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR - * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES - * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN - * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF - * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE. - * - */ - -#include "kernel/yosys.h" -#include "kernel/sigtools.h" - -USING_YOSYS_NAMESPACE -PRIVATE_NAMESPACE_BEGIN - -bool binary_pred(SigBit s1, SigBit s2) -{ - return s1 == s2; -} - -static void proc_glwren(Module *module) -{ - std::vector ram_cells; - - // Gather RAM cells - for (auto cell : module->cells()) - { - if (cell->type.in(ID(CC_DPSRAM_20K), ID(CC_DPSRAM_40K))) { - ram_cells.push_back(cell); - } - } - - // Convert bitmask signals - for (auto cell : ram_cells) - { - std::vector mska = cell->getPort(ID(A_BM)).to_sigbit_vector(); - std::vector mskb = cell->getPort(ID(B_BM)).to_sigbit_vector(); - - // Remove State::S0 from vectors - mska.erase(std::remove_if(mska.begin(), mska.end(), [](const SigBit & sb){return (sb == State::S0);}), mska.end()); - mskb.erase(std::remove_if(mskb.begin(), mskb.end(), [](const SigBit & sb){return (sb == State::S0);}), mskb.end()); - - if (mska.size() + mskb.size() == 0) { - break; - } - - if (cell->getParam(ID(RAM_MODE)).decode_string() == "SDP") - { - std::vector msk; - msk.insert(msk.end(), mska.begin(), mska.end()); - msk.insert(msk.end(), mskb.begin(), mskb.end()); - - if (std::equal(msk.begin() + 1, msk.end(), msk.begin(), binary_pred)) - { - if ((cell->getPort(ID(A_WE)) == State::S1) && (cell->getPort(ID(B_WE)) == State::S0)) - { - cell->setPort(ID(A_WE), msk[0]); - cell->setPort(ID(B_WE), Const(0, 1)); - // Set bitmask bits to _1_ - cell->setPort(ID(A_BM), Const(State::S1, mska.size())); - cell->setPort(ID(B_BM), Const(State::S1, mskb.size())); - } - } - } - else // RAM_MODE == "TDP" - { - if (std::equal(mska.begin() + 1, mska.end(), mska.begin(), binary_pred)) - { - if (cell->getPort(ID(A_WE)) == State::S1) - { - // Signals are all equal - cell->setPort(ID(A_WE), mska[0]); - cell->setPort(ID(A_BM), Const(State::S1, mska.size())); - } - } - if (std::equal(mskb.begin() + 1, mskb.end(), mskb.begin(), binary_pred)) - { - if (cell->getPort(ID(B_WE)) == State::S1) - { - cell->setPort(ID(B_WE), mskb[0]); - // Set bitmask bits to _1_ - cell->setPort(ID(B_BM), Const(State::S1, mskb.size())); - } - } - } - } -} - -struct GateMateBramOptPass : public Pass { - GateMateBramOptPass() : Pass("gatemate_bramopt", "GateMate: optimize block RAM cells") { } - void help() override - { - // |---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---| - log("\n"); - log(" gatemate_bramopt [options] [selection]\n"); - log("\n"); - log("This pass processes all CC_BRAM_20K and CC_BRAM_40K cells and tries to"); - log("convert its enable bitmask wires to a single global enable signal."); - log("\n"); - log(" -noglwren\n"); - log(" do not convert bitmasks to single global write enable signals.\n"); - log("\n"); - } - - bool noglwren; - - void clear_flags() override - { - noglwren = false; - } - - void execute(std::vector args, RTLIL::Design *design) override - { - log_header(design, "Executing GATEMATE_BRAMOPT pass (optimize block RAM).\n"); - - size_t argidx; - for (argidx = 1; argidx < args.size(); argidx++) - { - if (args[argidx] == "-noglwren") { - noglwren = true; - continue; - } - break; - } - extra_args(args, argidx, design); - - for (auto module : design->selected_modules()) - { - if (!noglwren) { - proc_glwren(module); - } - } - } -} GateMateBramOptPass; - -PRIVATE_NAMESPACE_END diff --git a/techlibs/gatemate/synth_gatemate.cc b/techlibs/gatemate/synth_gatemate.cc index b570e1e2a..978434c89 100644 --- a/techlibs/gatemate/synth_gatemate.cc +++ b/techlibs/gatemate/synth_gatemate.cc @@ -264,7 +264,6 @@ struct SynthGateMatePass : public ScriptPass "t:$__CC_BRAM_20K_TDP t:$__CC_BRAM_40K_TDP " ); run("techmap -map +/gatemate/brams_map.v"); - run("gatemate_bramopt"); } } -- 2.30.2