Revert "Add shregmap -init_msb_first and use in synth_xilinx"
authorEddie Hung <eddieh@ece.ubc.ca>
Thu, 14 Mar 2019 16:01:48 +0000 (09:01 -0700)
committerEddie Hung <eddieh@ece.ubc.ca>
Thu, 14 Mar 2019 16:01:48 +0000 (09:01 -0700)
This reverts commit 26ecbc1aee1dca1c186ab2b51835d74f67bc3e75.

passes/techmap/shregmap.cc
techlibs/xilinx/synth_xilinx.cc

index 6cd9082dcf385de0a2335d20618a7b8efec923b6..f20863ba0f7cf34425c186da292021050cf42b19 100644 (file)
@@ -34,7 +34,7 @@ struct ShregmapOptions
 {
        int minlen, maxlen;
        int keep_before, keep_after;
-       bool zinit, init, params, ffe, init_msb_first;
+       bool zinit, init, params, ffe;
        dict<IdString, pair<IdString, IdString>> ffcells;
        ShregmapTech *tech;
 
@@ -48,7 +48,6 @@ struct ShregmapOptions
                init = false;
                params = false;
                ffe = false;
-               init_msb_first = false;
                tech = nullptr;
        }
 };
@@ -308,8 +307,6 @@ struct ShregmapWorker
                                                initval.push_back(State::S0);
                                        remove_init.insert(bit);
                                }
-                               if (opts.init_msb_first)
-                                       std::reverse(initval.begin(), initval.end());
                                first_cell->setParam("\\INIT", initval);
                        }
 
@@ -445,13 +442,9 @@ struct ShregmapPass : public Pass {
                log("\n");
                log("    -init\n");
                log("        map initialized registers to the shift reg, add an INIT parameter to\n");
-               log("        generated cells with the initialization value. (First bit to shift out\n");
+               log("        generated cells with the initialization value. (first bit to shift out\n");
                log("        in LSB position)\n");
                log("\n");
-               log("    -init_msb_first\n");
-               log("        same as -init, but INIT parameter to have first bit to shift out\n");
-               log("        in MSB position.\n");
-               log("\n");
                log("    -tech greenpak4\n");
                log("        map to greenpak4 shift registers.\n");
                log("\n");
@@ -522,11 +515,6 @@ struct ShregmapPass : public Pass {
                                opts.init = true;
                                continue;
                        }
-                       if (args[argidx] == "-init_msb_first") {
-                               opts.init = true;
-                               opts.init_msb_first = true;
-                               continue;
-                       }
                        if (args[argidx] == "-params") {
                                opts.params = true;
                                continue;
index 169f3b7cec7fe94c85bc357e05a7edce72ca75f6..1978ccf21a8f8974998b891a83bdf7a3259d6aea 100644 (file)
@@ -111,7 +111,7 @@ struct SynthXilinxPass : public Pass
                log("        dff2dffe\n");
                log("        opt -full\n");
                log("        techmap -map +/techmap.v -map +/xilinx/arith_map.v\n");
-               log("        shregmap -init_msb_first -params -enpol any_or_none\n");
+               log("        shregmap -init -params -enpol any_or_none\n");
                log("        techmap -map +/xilinx/ff_map.v\n");
                log("        opt -fast\n");
                log("\n");
@@ -262,9 +262,8 @@ struct SynthXilinxPass : public Pass
                                Pass::call(design, "techmap -map +/techmap.v -map +/xilinx/arith_map.v");
                        }
 
-                       Pass::call(design, "shregmap -init_msb_first -params -enpol any_or_none");
+                       Pass::call(design, "shregmap -initt -params -enpol any_or_none");
                        Pass::call(design, "techmap -map +/xilinx/ff_map.v");
-                       Pass::call(design, "hierarchy -check");
                        Pass::call(design, "opt -fast");
                }