Revert "Merge branch 'eddie/split_shiftx' into xc7mux"
authorEddie Hung <eddie@fpgeh.com>
Fri, 26 Apr 2019 22:32:02 +0000 (15:32 -0700)
committerEddie Hung <eddie@fpgeh.com>
Fri, 26 Apr 2019 22:32:02 +0000 (15:32 -0700)
This reverts commit 3042d5833041021bb45252b0cc862e9eff3d27d3, reversing
changes made to feff9764540cbf1152459cb377fc68d8e10c7153.

README.md
kernel/cellaigs.cc
kernel/celltypes.h
passes/opt/wreduce.cc
passes/pmgen/.gitignore
passes/pmgen/Makefile.inc
passes/pmgen/README.md
passes/pmgen/split_shiftx.cc [deleted file]
passes/pmgen/split_shiftx.pmg [deleted file]

index 913777f2ee463933d82573fc6de33aed488e8447..46bed42426d8f3def0ab4e5079e37aab1aec29cb 100644 (file)
--- a/README.md
+++ b/README.md
@@ -370,7 +370,7 @@ Verilog Attributes and non-standard features
 
 - When defining a macro with `define, all text between triple double quotes
   is interpreted as macro body, even if it contains unescaped newlines. The
-  triple double quotes are removed from the macro body. For example:
+  tipple double quotes are removed from the macro body. For example:
 
       `define MY_MACRO(a, b) """
          assign a = 23;
@@ -457,7 +457,7 @@ Non-standard or SystemVerilog features for formal verification
   supported in any clocked block.
 
 - The syntax ``@($global_clock)`` can be used to create FFs that have no
-  explicit clock input (``$ff`` cells). The same can be achieved by using
+  explicit clock input ($ff cells). The same can be achieved by using
   ``@(posedge <netname>)`` or ``@(negedge <netname>)`` when ``<netname>``
   is marked with the ``(* gclk *)`` Verilog attribute.
 
@@ -470,7 +470,7 @@ from SystemVerilog:
 
 - The ``assert`` statement from SystemVerilog is supported in its most basic
   form. In module context: ``assert property (<expression>);`` and within an
-  always block: ``assert(<expression>);``. It is transformed to an ``$assert`` cell.
+  always block: ``assert(<expression>);``. It is transformed to a $assert cell.
 
 - The ``assume``, ``restrict``, and ``cover`` statements from SystemVerilog are
   also supported. The same limitations as with the ``assert`` statement apply.
index 26c625f89a62d2f3ab1f18e3fce70eba82664333..5fd76afe53954615d7d50670281939d86ca2a769 100644 (file)
@@ -453,7 +453,7 @@ Aig::Aig(Cell *cell)
                int B = mk.inport("\\B");
                int C = mk.inport("\\C");
                int D = mk.inport("\\D");
-               int Y = mk.nand_gate(mk.or_gate(A, B), mk.or_gate(C, D));
+               int Y = mk.nand_gate(mk.nor_gate(A, B), mk.nor_gate(C, D));
                mk.outport(Y, "\\Y");
                goto optimize;
        }
index 0da78c313e90defb9e3bc0db252aabf43c52c4d3..ae88f4aaf90e0e114a0bb31bc803f45bd44f7948 100644 (file)
@@ -464,7 +464,7 @@ struct CellTypes
                if (cell->type == "$_AOI4_")
                        return eval_not(const_or(const_and(arg1, arg2, false, false, 1), const_and(arg3, arg4, false, false, 1), false, false, 1));
                if (cell->type == "$_OAI4_")
-                       return eval_not(const_and(const_or(arg1, arg2, false, false, 1), const_or(arg3, arg4, false, false, 1), false, false, 1));
+                       return eval_not(const_and(const_or(arg1, arg2, false, false, 1), const_and(arg3, arg4, false, false, 1), false, false, 1));
 
                log_assert(arg4.bits.size() == 0);
                return eval(cell, arg1, arg2, arg3, errp);
index 68e077cf9eb5e1249f9802494aa06e2fe4987c85..52245ce3e4f9bf21e924be6128ecce36d9d64d49 100644 (file)
@@ -462,10 +462,12 @@ struct WreduceWorker
                                        SigSpec initsig = init_attr_sigmap(w);
                                        int width = std::min(GetSize(initval), GetSize(initsig));
                                        for (int i = 0; i < width; i++) {
+                                               log_dump(initsig[i], remove_init_bits.count(initsig[i]));
                                                if (!remove_init_bits.count(initsig[i]))
                                                        new_initval[i] = initval[i];
                                        }
                                        w->attributes.at("\\init") = new_initval;
+                                       log_dump(w->name, initval, new_initval);
                                }
                        }
                }
index 52dfd93f370de104b7690811ae1f96b7b30ef924..c9263057e5c5d004ab0df7d834fb78ca5ae04083 100644 (file)
@@ -1 +1 @@
-*_pm.h
+/ice40_dsp_pm.h
index 5669bd3d1cd57d099ae14ebf37e9bb21c9d39ace..e0609d9ba368e1f76e0092bbb0cd60ff63ff4927 100644 (file)
@@ -1,11 +1,8 @@
-PMG_SRC = $(wildcard passes/pmgen/*.pmg)
-PMG_OBJS += $(patsubst %.pmg, %.o, $(PMG_SRC))
-OBJS += $(PMG_OBJS)
+OBJS += passes/pmgen/ice40_dsp.o
 
-$(PMG_OBJS): %.o: %_pm.h
+passes/pmgen/ice40_dsp.o: passes/pmgen/ice40_dsp_pm.h
+EXTRA_OBJS += passes/pmgen/ice40_dsp_pm.h
+.SECONDARY: passes/pmgen/ice40_dsp_pm.h
 
-EXTRA_OBJS += $(patsubst %.pmg, %_pm.h, $(PMG_SRC))
-.SECONDARY: $(EXTRA_OBJS)
-
-%_pm.h: passes/pmgen/pmgen.py %.pmg
+passes/pmgen/ice40_dsp_pm.h: passes/pmgen/pmgen.py passes/pmgen/ice40_dsp.pmg
        $(P) mkdir -p passes/pmgen && python3 $^ $@
index 320e95a7777e3411849b0a353052c360f43136eb..7a46558b12c596d35a15e43bcaf1543bdb68ed25 100644 (file)
@@ -220,5 +220,5 @@ But in some cases it is more natural to utilize the implicit branch statement:
         portAB = \B;
     endcode
 
-There is an implicit `code..endcode` block at the end of each `.pmg` file
+There is an implicit `code..endcode` block at the end of each `.pgm` file
 that just accepts everything that gets all the way there.
diff --git a/passes/pmgen/split_shiftx.cc b/passes/pmgen/split_shiftx.cc
deleted file mode 100644 (file)
index 3cbabcd..0000000
+++ /dev/null
@@ -1,84 +0,0 @@
-/*
- *  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.
- *
- */
-
-#include "kernel/yosys.h"
-#include "kernel/sigtools.h"
-#include "passes/pmgen/split_shiftx_pm.h"
-
-USING_YOSYS_NAMESPACE
-PRIVATE_NAMESPACE_BEGIN
-
-void create_split_shiftx(split_shiftx_pm &pm)
-{
-       log_assert(pm.st.shiftx);
-       if (pm.blacklist_cells.count(pm.st.shiftx))
-               return;
-       SigSpec A = pm.st.shiftx->getPort("\\A");
-       SigSpec B = pm.st.shiftxB;
-       log_assert(!B.empty());
-       SigSpec Y = pm.st.shiftx->getPort("\\Y");
-       const int A_WIDTH = pm.st.shiftx->getParam("\\A_WIDTH").as_int();
-       const int B_WIDTH = GetSize(pm.st.shiftxB);
-       const int Y_WIDTH = pm.st.shiftx->getParam("\\Y_WIDTH").as_int();
-       int trailing_zeroes = 0;
-       for (; B[trailing_zeroes] == RTLIL::S0; ++trailing_zeroes) ;
-       const int WIDTH = trailing_zeroes > 0 ? 1 << trailing_zeroes : Y_WIDTH;
-       std::vector<SigBit> bits;
-       bits.resize(A_WIDTH / WIDTH);
-       for (int i = 0; i < Y_WIDTH; ++i) {
-               for (int j = 0; j < A_WIDTH/WIDTH; ++j)
-                       bits[j] = A[j*WIDTH + i];
-               pm.module->addShiftx(NEW_ID, bits, B.extract(trailing_zeroes, B_WIDTH-trailing_zeroes), Y[i]);
-       }
-       pm.st.shiftx->unsetPort("\\Y");
-
-       pm.autoremove(pm.st.shiftx);
-       pm.autoremove(pm.st.macc);
-}
-
-struct BitblastShiftxPass : public Pass {
-       BitblastShiftxPass() : Pass("split_shiftx", "Split up multi-bit $shiftx cells") { }
-       void help() YS_OVERRIDE
-       {
-               //   |---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---|---v---|
-               log("\n");
-               log("    split_shiftx [selection]\n");
-               log("\n");
-               log("Split up $shiftx cells where Y_WIDTH > 1, with consideration for any $macc\n");
-               log("cells -- configured as a constant multiplier equal to Y_WIDTH -- that may be\n");
-               log("driving their B inputs.\n");
-               log("\n");
-       }
-       void execute(std::vector<std::string> args, RTLIL::Design *design) YS_OVERRIDE
-       {
-               log_header(design, "Executing SPLIT_SHIFTX pass.\n");
-
-               size_t argidx;
-               for (argidx = 1; argidx < args.size(); argidx++)
-               {
-                       break;
-               }
-               extra_args(args, argidx, design);
-
-               for (auto module : design->selected_modules())
-                       split_shiftx_pm(module, module->selected_cells()).run(create_split_shiftx);
-       }
-} BitblastShiftxPass;
-
-PRIVATE_NAMESPACE_END
diff --git a/passes/pmgen/split_shiftx.pmg b/passes/pmgen/split_shiftx.pmg
deleted file mode 100644 (file)
index 3aafe19..0000000
+++ /dev/null
@@ -1,59 +0,0 @@
-state <SigSpec> shiftxB
-
-match shiftx
-       select shiftx->type == $shiftx
-       select param(shiftx, \Y_WIDTH).as_int() > 1
-endmatch
-
-match macc
-       select macc->type == $macc
-       select param(macc, \B_WIDTH).as_int() == 0
-       optional
-endmatch
-
-code shiftxB
-       shiftxB = port(shiftx, \B);
-
-       if (macc) {
-               const int b_width = param(shiftx, \B_WIDTH).as_int();
-               if (param(shiftx, \B_SIGNED) != 0 && shiftxB[b_width-1] == RTLIL::S0)
-                       shiftxB = shiftxB.extract(0, b_width-1);
-
-               if (port(macc, \Y) != shiftxB) {
-                       blacklist(shiftx);
-                       reject;
-               }
-
-               Const config = param(macc, \CONFIG);
-               const int config_width = param(macc, \CONFIG_WIDTH).as_int();
-               const int num_bits = config.extract(0, 4).as_int();
-               const int num_ports = (config_width - 4) / (2 + 2*num_bits);
-               if (num_ports != 1) {
-                       blacklist(shiftx);
-                       reject;
-               }
-               // IS_SIGNED?
-               if (config[4] == 1) {
-                       blacklist(shiftx);
-                       reject;
-               }
-               // DO_SUBTRACT?
-               if (config[5] == 1) {
-                       blacklist(shiftx);
-                       reject;
-               }
-               const int port_size_A = config.extract(6, num_bits).as_int();
-               const int port_size_B = config.extract(6 + num_bits, num_bits).as_int();
-               const SigSpec port_B = port(macc, \A).extract(port_size_A, port_size_B);
-               if (!port_B.is_fully_const()) {
-                       blacklist(shiftx);
-                       reject;
-               }
-               const int multiply_factor = port_B.as_int();
-               if (multiply_factor != param(shiftx, \Y_WIDTH).as_int()) {
-                       blacklist(shiftx);
-                       reject;
-               }
-               shiftxB = port(macc, \A).extract(0, port_size_A);
-       }
-endcode