Move GENERATE_PATTERN macro to separate utility header
authorClifford Wolf <clifford@clifford.at>
Wed, 16 Oct 2019 09:40:01 +0000 (11:40 +0200)
committerClifford Wolf <clifford@clifford.at>
Wed, 16 Oct 2019 09:40:01 +0000 (11:40 +0200)
Signed-off-by: Clifford Wolf <clifford@clifford.at>
passes/pmgen/generate.h [new file with mode: 0644]
passes/pmgen/pmgen.py
passes/pmgen/test_pmgen.cc

diff --git a/passes/pmgen/generate.h b/passes/pmgen/generate.h
new file mode 100644 (file)
index 0000000..354583d
--- /dev/null
@@ -0,0 +1,140 @@
+/*
+ *  yosys -- Yosys Open SYnthesis Suite
+ *
+ *  Copyright (C) 2012  Clifford Wolf <clifford@clifford.at>
+ *
+ *  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.
+ *
+ */
+
+#ifndef PMGEN_GENERATE
+#define PMGEN_GENERATE
+
+#define GENERATE_PATTERN(pmclass, pattern) \
+       generate_pattern<pmclass>([](pmclass &pm, std::function<void()> f){ return pm.run_ ## pattern(f); }, #pmclass, #pattern, design)
+
+void pmtest_addports(Module *module)
+{
+       pool<SigBit> driven_bits, used_bits;
+       SigMap sigmap(module);
+       int icnt = 0, ocnt = 0;
+
+       for (auto cell : module->cells())
+       for (auto conn : cell->connections())
+       {
+               if (cell->input(conn.first))
+                       for (auto bit : sigmap(conn.second))
+                               used_bits.insert(bit);
+               if (cell->output(conn.first))
+                       for (auto bit : sigmap(conn.second))
+                               driven_bits.insert(bit);
+       }
+
+       for (auto wire : vector<Wire*>(module->wires()))
+       {
+               SigSpec ibits, obits;
+               for (auto bit : sigmap(wire)) {
+                       if (!used_bits.count(bit))
+                               obits.append(bit);
+                       if (!driven_bits.count(bit))
+                               ibits.append(bit);
+               }
+               if (!ibits.empty()) {
+                       Wire *w = module->addWire(stringf("\\i%d", icnt++), GetSize(ibits));
+                       w->port_input = true;
+                       module->connect(ibits, w);
+               }
+               if (!obits.empty()) {
+                       Wire *w = module->addWire(stringf("\\o%d", ocnt++), GetSize(obits));
+                       w->port_output = true;
+                       module->connect(w, obits);
+               }
+       }
+
+       module->fixup_ports();
+}
+
+template <class pm>
+void generate_pattern(std::function<void(pm&,std::function<void()>)> run, const char *pmclass, const char *pattern, Design *design)
+{
+       log("Generating \"%s\" patterns for pattern matcher \"%s\".\n", pattern, pmclass);
+
+       int modcnt = 0;
+       int maxmodcnt = 100;
+       int maxsubcnt = 4;
+       int timeout = 0;
+       vector<Module*> mods;
+
+       while (modcnt < maxmodcnt)
+       {
+               int submodcnt = 0, itercnt = 0, cellcnt = 0;
+               Module *mod = design->addModule(NEW_ID);
+
+               while (modcnt < maxmodcnt && submodcnt < maxsubcnt && itercnt++ < 1000)
+               {
+                       if (timeout++ > 10000)
+                               log_error("pmgen generator is stuck: 10000 iterations with no matching module generated.\n");
+
+                       pm matcher(mod, mod->cells());
+
+                       matcher.rng(1);
+                       matcher.rngseed += modcnt;
+                       matcher.rng(1);
+                       matcher.rngseed += submodcnt;
+                       matcher.rng(1);
+                       matcher.rngseed += itercnt;
+                       matcher.rng(1);
+                       matcher.rngseed += cellcnt;
+                       matcher.rng(1);
+
+                       if (GetSize(mod->cells()) != cellcnt)
+                       {
+                               bool found_match = false;
+                               run(matcher, [&](){ found_match = true; });
+                               cellcnt = GetSize(mod->cells());
+
+                               if (found_match) {
+                                       Module *m = design->addModule(stringf("\\pmtest_%s_%s_%05d",
+                                                       pmclass, pattern, modcnt++));
+                                       log("Creating module %s with %d cells.\n", log_id(m), cellcnt);
+                                       mod->cloneInto(m);
+                                       pmtest_addports(m);
+                                       mods.push_back(m);
+                                       submodcnt++;
+                                       timeout = 0;
+                               }
+                       }
+
+                       matcher.generate_mode = true;
+                       run(matcher, [](){});
+               }
+
+               if (submodcnt && maxsubcnt < (1 << 16))
+                       maxsubcnt *= 2;
+
+               design->remove(mod);
+       }
+
+       Module *m = design->addModule(stringf("\\pmtest_%s_%s", pmclass, pattern));
+       log("Creating module %s with %d cells.\n", log_id(m), GetSize(mods));
+       for (auto mod : mods) {
+               Cell *c = m->addCell(mod->name, mod->name);
+               for (auto port : mod->ports) {
+                       Wire *w = m->addWire(NEW_ID, GetSize(mod->wire(port)));
+                       c->setPort(port, w);
+               }
+       }
+       pmtest_addports(m);
+}
+
+#endif
index 39a09991d04b8f901c6a14c09d00ebb1547bf987..df0ffaff287a3d7dc0cab0d9c17f1190f31e55b3 100644 (file)
@@ -362,6 +362,7 @@ with open(outfile, "w") as f:
     print("  Module *module;", file=f)
     print("  SigMap sigmap;", file=f)
     print("  std::function<void()> on_accept;", file=f)
+    print("  bool setup_done;", file=f)
     print("  bool generate_mode;", file=f)
     print("  int accept_cnt;", file=f)
     print("", file=f)
@@ -477,7 +478,17 @@ with open(outfile, "w") as f:
     print("", file=f)
 
     print("  {}_pm(Module *module, const vector<Cell*> &cells) :".format(prefix), file=f)
-    print("      module(module), sigmap(module), generate_mode(false), rngseed(12345678) {", file=f)
+    print("      module(module), sigmap(module), setup_done(false), generate_mode(false), rngseed(12345678) {", file=f)
+    print("    setup(cells);", file=f)
+    print("  }", file=f)
+    print("", file=f)
+
+    print("  {}_pm(Module *module) :".format(prefix), file=f)
+    print("      module(module), sigmap(module), setup_done(false), generate_mode(false), rngseed(12345678) {", file=f)
+    print("  }", file=f)
+    print("", file=f)
+
+    print("  void setup(const vector<Cell*> &cells) {", file=f)
     for current_pattern in sorted(patterns.keys()):
         for s, t in sorted(udata_types[current_pattern].items()):
             if t.endswith("*"):
@@ -485,6 +496,8 @@ with open(outfile, "w") as f:
             else:
                 print("    ud_{}.{} = {}();".format(current_pattern, s, t), file=f)
     current_pattern = None
+    print("    log_assert(!setup_done);", file=f)
+    print("    setup_done = true;", file=f)
     print("    for (auto port : module->ports)", file=f)
     print("      add_siguser(module->wire(port), nullptr);", file=f)
     print("    for (auto cell : module->cells())", file=f)
@@ -539,6 +552,7 @@ with open(outfile, "w") as f:
 
     for current_pattern in sorted(patterns.keys()):
         print("  int run_{}(std::function<void()> on_accept_f) {{".format(current_pattern), file=f)
+        print("    log_assert(setup_done);", file=f)
         print("    accept_cnt = 0;", file=f)
         print("    on_accept = on_accept_f;", file=f)
         print("    rollback = 0;", file=f)
index 4f3eec935ac7b280315c460c9e8bcaf6aa7a8690..72dc18dcce0b820e9ff5d7bce887a7521681f80e 100644 (file)
 USING_YOSYS_NAMESPACE
 PRIVATE_NAMESPACE_BEGIN
 
-// for peepopt_pm
-bool did_something;
-
 #include "passes/pmgen/test_pmgen_pm.h"
 #include "passes/pmgen/ice40_dsp_pm.h"
 #include "passes/pmgen/xilinx_srl_pm.h"
-#include "passes/pmgen/peepopt_pm.h"
+
+#include "generate.h"
 
 void reduce_chain(test_pmgen_pm &pm)
 {
@@ -118,123 +116,6 @@ void opt_eqpmux(test_pmgen_pm &pm)
        log("    -> %s (%s)\n", log_id(c), log_id(c->type));
 }
 
-#define GENERATE_PATTERN(pmclass, pattern) \
-       generate_pattern<pmclass>([](pmclass &pm, std::function<void()> f){ return pm.run_ ## pattern(f); }, #pmclass, #pattern, design)
-
-void pmtest_addports(Module *module)
-{
-       pool<SigBit> driven_bits, used_bits;
-       SigMap sigmap(module);
-       int icnt = 0, ocnt = 0;
-
-       for (auto cell : module->cells())
-       for (auto conn : cell->connections())
-       {
-               if (cell->input(conn.first))
-                       for (auto bit : sigmap(conn.second))
-                               used_bits.insert(bit);
-               if (cell->output(conn.first))
-                       for (auto bit : sigmap(conn.second))
-                               driven_bits.insert(bit);
-       }
-
-       for (auto wire : vector<Wire*>(module->wires()))
-       {
-               SigSpec ibits, obits;
-               for (auto bit : sigmap(wire)) {
-                       if (!used_bits.count(bit))
-                               obits.append(bit);
-                       if (!driven_bits.count(bit))
-                               ibits.append(bit);
-               }
-               if (!ibits.empty()) {
-                       Wire *w = module->addWire(stringf("\\i%d", icnt++), GetSize(ibits));
-                       w->port_input = true;
-                       module->connect(ibits, w);
-               }
-               if (!obits.empty()) {
-                       Wire *w = module->addWire(stringf("\\o%d", ocnt++), GetSize(obits));
-                       w->port_output = true;
-                       module->connect(w, obits);
-               }
-       }
-
-       module->fixup_ports();
-}
-
-template <class pm>
-void generate_pattern(std::function<void(pm&,std::function<void()>)> run, const char *pmclass, const char *pattern, Design *design)
-{
-       log("Generating \"%s\" patterns for pattern matcher \"%s\".\n", pattern, pmclass);
-
-       int modcnt = 0;
-       int maxmodcnt = 100;
-       int maxsubcnt = 4;
-       int timeout = 0;
-       vector<Module*> mods;
-
-       while (modcnt < maxmodcnt)
-       {
-               int submodcnt = 0, itercnt = 0, cellcnt = 0;
-               Module *mod = design->addModule(NEW_ID);
-
-               while (modcnt < maxmodcnt && submodcnt < maxsubcnt && itercnt++ < 1000)
-               {
-                       if (timeout++ > 10000)
-                               log_error("pmgen generator is stuck: 10000 iterations with no matching module generated.\n");
-
-                       pm matcher(mod, mod->cells());
-
-                       matcher.rng(1);
-                       matcher.rngseed += modcnt;
-                       matcher.rng(1);
-                       matcher.rngseed += submodcnt;
-                       matcher.rng(1);
-                       matcher.rngseed += itercnt;
-                       matcher.rng(1);
-                       matcher.rngseed += cellcnt;
-                       matcher.rng(1);
-
-                       if (GetSize(mod->cells()) != cellcnt)
-                       {
-                               bool found_match = false;
-                               run(matcher, [&](){ found_match = true; });
-                               cellcnt = GetSize(mod->cells());
-
-                               if (found_match) {
-                                       Module *m = design->addModule(stringf("\\pmtest_%s_%s_%05d",
-                                                       pmclass, pattern, modcnt++));
-                                       log("Creating module %s with %d cells.\n", log_id(m), cellcnt);
-                                       mod->cloneInto(m);
-                                       pmtest_addports(m);
-                                       mods.push_back(m);
-                                       submodcnt++;
-                                       timeout = 0;
-                               }
-                       }
-
-                       matcher.generate_mode = true;
-                       run(matcher, [](){});
-               }
-
-               if (submodcnt && maxsubcnt < (1 << 16))
-                       maxsubcnt *= 2;
-
-               design->remove(mod);
-       }
-
-       Module *m = design->addModule(stringf("\\pmtest_%s_%s", pmclass, pattern));
-       log("Creating module %s with %d cells.\n", log_id(m), GetSize(mods));
-       for (auto mod : mods) {
-               Cell *c = m->addCell(mod->name, mod->name);
-               for (auto port : mod->ports) {
-                       Wire *w = m->addWire(NEW_ID, GetSize(mod->wire(port)));
-                       c->setPort(port, w);
-               }
-       }
-       pmtest_addports(m);
-}
-
 struct TestPmgenPass : public Pass {
        TestPmgenPass() : Pass("test_pmgen", "test pass for pmgen") { }
        void help() YS_OVERRIDE
@@ -355,12 +236,6 @@ struct TestPmgenPass : public Pass {
                if (pattern == "xilinx_srl.variable")
                        return GENERATE_PATTERN(xilinx_srl_pm, variable);
 
-               if (pattern == "peepopt-muldiv")
-                       return GENERATE_PATTERN(peepopt_pm, muldiv);
-
-               if (pattern == "peepopt-shiftmul")
-                       return GENERATE_PATTERN(peepopt_pm, shiftmul);
-
                log_cmd_error("Unknown pattern: %s\n", pattern.c_str());
        }