bugpoint: add -assigns and -updates options.
authorwhitequark <whitequark@whitequark.org>
Tue, 9 Jul 2019 09:08:38 +0000 (09:08 +0000)
committerwhitequark <whitequark@whitequark.org>
Tue, 9 Jul 2019 09:27:43 +0000 (09:27 +0000)
passes/cmds/bugpoint.cc

index 038ab7c7c923b9023f8d6b50f5d6efa7797e28d6..5a47988ec389f6d4c153dba48992bff1b33da7cd 100644 (file)
@@ -51,14 +51,14 @@ struct BugpointPass : public Pass {
                log("        only consider crashes that place this string in the log file.\n");
                log("\n");
                log("    -fast\n");
-               log("        run `clean -purge` after each minimization step. converges faster, but\n");
-               log("        produces larger testcases, and may fail to produce any testcase at all if\n");
-               log("        the crash is related to dangling wires.\n");
+               log("        run `proc_clean; clean -purge` after each minimization step. converges\n");
+               log("        faster, but produces larger testcases, and may fail to produce any\n");
+               log("        testcase at all if the crash is related to dangling wires.\n");
                log("\n");
                log("    -clean\n");
-               log("        run `clean -purge` before checking testcase and after finishing. produces\n");
-               log("        smaller and more useful testcases, but may fail to produce any testcase\n");
-               log("        at all if the crash is related to dangling wires.\n");
+               log("        run `proc_clean; clean -purge` before checking testcase and after\n");
+               log("        finishing. produces smaller and more useful testcases, but may fail to\n");
+               log("        produce any testcase at all if the crash is related to dangling wires.\n");
                log("\n");
                log("    -modules\n");
                log("        try to remove modules.\n");
@@ -72,6 +72,12 @@ struct BugpointPass : public Pass {
                log("    -connections\n");
                log("        try to reconnect ports to 'x.\n");
                log("\n");
+               log("    -assigns\n");
+               log("        try to remove process assigns from cases.\n");
+               log("\n");
+               log("    -updates\n");
+               log("        try to remove process updates from syncs.\n");
+               log("\n");
        }
 
        bool run_yosys(RTLIL::Design *design, string yosys_cmd, string script)
@@ -110,6 +116,7 @@ struct BugpointPass : public Pass {
                RTLIL::Design *design_copy = new RTLIL::Design;
                for (auto &it : design->modules_)
                        design_copy->add(it.second->clone());
+               Pass::call(design_copy, "proc_clean -quiet");
                Pass::call(design_copy, "clean -purge");
 
                if (do_delete)
@@ -117,7 +124,7 @@ struct BugpointPass : public Pass {
                return design_copy;
        }
 
-       RTLIL::Design *simplify_something(RTLIL::Design *design, int &seed, bool stage2, bool modules, bool ports, bool cells, bool connections)
+       RTLIL::Design *simplify_something(RTLIL::Design *design, int &seed, bool stage2, bool modules, bool ports, bool cells, bool connections, bool assigns, bool updates)
        {
                RTLIL::Design *design_copy = new RTLIL::Design;
                for (auto &it : design->modules_)
@@ -225,6 +232,59 @@ struct BugpointPass : public Pass {
                                }
                        }
                }
+               if (assigns)
+               {
+                       for (auto mod : design_copy->modules())
+                       {
+                               if (mod->get_blackbox_attribute())
+                                       continue;
+
+                               for (auto &pr : mod->processes)
+                               {
+                                       vector<RTLIL::CaseRule*> cases = {&pr.second->root_case};
+                                       while (!cases.empty())
+                                       {
+                                               RTLIL::CaseRule *cs = cases[0];
+                                               cases.erase(cases.begin());
+                                               for (auto it = cs->actions.begin(); it != cs->actions.end(); ++it)
+                                               {
+                                                       if (index++ == seed)
+                                                       {
+                                                               log("Trying to remove assign %s %s in %s.%s.\n", log_signal((*it).first), log_signal((*it).second), mod->name.c_str(), pr.first.c_str());
+                                                               cs->actions.erase(it);
+                                                               return design_copy;
+                                                       }
+                                               }
+                                               for (auto &sw : cs->switches)
+                                                       cases.insert(cases.end(), sw->cases.begin(), sw->cases.end());
+                                       }
+                               }
+                       }
+               }
+               if (updates)
+               {
+                       for (auto mod : design_copy->modules())
+                       {
+                               if (mod->get_blackbox_attribute())
+                                       continue;
+
+                               for (auto &pr : mod->processes)
+                               {
+                                       for (auto &sy : pr.second->syncs)
+                                       {
+                                               for (auto it = sy->actions.begin(); it != sy->actions.end(); ++it)
+                                               {
+                                                       if (index++ == seed)
+                                                       {
+                                                               log("Trying to remove sync %s update %s %s in %s.%s.\n", log_signal(sy->signal), log_signal((*it).first), log_signal((*it).second), mod->name.c_str(), pr.first.c_str());
+                                                               sy->actions.erase(it);
+                                                               return design_copy;
+                                                       }
+                                               }
+                                       }
+                               }
+                       }
+               }
                return NULL;
        }
 
@@ -232,7 +292,7 @@ struct BugpointPass : public Pass {
        {
                string yosys_cmd = "yosys", script, grep;
                bool fast = false, clean = false;
-               bool modules = false, ports = false, cells = false, connections = false, has_part = false;
+               bool modules = false, ports = false, cells = false, connections = false, assigns = false, updates = false, has_part = false;
 
                size_t argidx;
                for (argidx = 1; argidx < args.size(); argidx++)
@@ -277,6 +337,16 @@ struct BugpointPass : public Pass {
                                has_part = true;
                                continue;
                        }
+                       if (args[argidx] == "-assigns") {
+                               assigns = true;
+                               has_part = true;
+                               continue;
+                       }
+                       if (args[argidx] == "-updates") {
+                               updates = true;
+                               has_part = true;
+                               continue;
+                       }
                        break;
                }
                extra_args(args, argidx, design);
@@ -290,6 +360,8 @@ struct BugpointPass : public Pass {
                        ports = true;
                        cells = true;
                        connections = true;
+                       assigns = true;
+                       updates = true;
                }
 
                if (!design->full_selection())
@@ -305,7 +377,7 @@ struct BugpointPass : public Pass {
                bool found_something = false, stage2 = false;
                while (true)
                {
-                       if (RTLIL::Design *simplified = simplify_something(crashing_design, seed, stage2, modules, ports, cells, connections))
+                       if (RTLIL::Design *simplified = simplify_something(crashing_design, seed, stage2, modules, ports, cells, connections, assigns, updates))
                        {
                                simplified = clean_design(simplified, fast, /*do_delete=*/true);