From: Eddie Hung <eddie@fpgeh.com>
Date: Tue, 17 Dec 2019 00:48:31 +0000 (-0800)
Subject: Put $__ABC9_{FF_,ASYNC} into same clock domain as abc9_flop
X-Git-Tag: working-ls180~881^2^2~76
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d9bf7061cd6a5efc20c053bc96d05650f0fa3a14;p=yosys.git

Put $__ABC9_{FF_,ASYNC} into same clock domain as abc9_flop
---

diff --git a/passes/techmap/abc9.cc b/passes/techmap/abc9.cc
index 8f2d45b62..34e122e7b 100644
--- a/passes/techmap/abc9.cc
+++ b/passes/techmap/abc9.cc
@@ -1095,7 +1095,7 @@ struct Abc9Pass : public Pass {
 			std::map<RTLIL::Cell*, std::set<RTLIL::SigBit>> cell_to_bit, cell_to_bit_up, cell_to_bit_down;
 			std::map<RTLIL::SigBit, std::set<RTLIL::Cell*>> bit_to_cell, bit_to_cell_up, bit_to_cell_down;
 
-			for (auto cell : all_cells) {
+			for (auto cell : all_cells)
 				for (auto &conn : cell->connections())
 				for (auto bit : assign_map(conn.second))
 					if (bit.wire != nullptr) {
@@ -1111,6 +1111,7 @@ struct Abc9Pass : public Pass {
 						}
 					}
 
+			for (auto cell : all_cells) {
 				auto inst_module = design->module(cell->type);
 				if (!inst_module || !inst_module->attributes.count("\\abc9_flop"))
 					continue;
@@ -1121,10 +1122,7 @@ struct Abc9Pass : public Pass {
 				SigSpec abc9_clock = assign_map(abc9_clock_wire);
 
 				unassigned_cells.erase(cell);
-				expand_queue.insert(cell);
 				expand_queue_up.insert(cell);
-				expand_queue_down.insert(cell);
-
 				clkdomain_t key(abc9_clock, cell->type);
 				assigned_cells[key].insert(cell->name);
 				assigned_cells_reverse[cell] = key;
@@ -1141,6 +1139,30 @@ struct Abc9Pass : public Pass {
 				    log_error("'%s.$abc9_init' is not a constant wire present in module '%s'.\n", cell->name.c_str(), log_id(module));
 				r2 = cell->attributes.insert(std::make_pair(ID(abc9_init), abc9_init.as_const()));
 				log_assert(r2.second);
+
+				// Also assign these special ABC9 cells to the
+				//   same clock domain
+				for (auto b : cell_to_bit_down[cell])
+				for (auto c : bit_to_cell_down[b])
+					if (c->type == "$__ABC9_FF_") {
+						cell = c;
+						unassigned_cells.erase(cell);
+						assigned_cells[key].insert(cell->name);
+						assigned_cells_reverse[cell] = key;
+						break;
+					}
+				for (auto b : cell_to_bit_down[cell])
+				for (auto c : bit_to_cell_down[b])
+					if (c->type == "$__ABC9_ASYNC") {
+						cell = c;
+						unassigned_cells.erase(cell);
+						assigned_cells[key].insert(cell->name);
+						assigned_cells_reverse[cell] = key;
+						break;
+					}
+
+				expand_queue.insert(cell);
+				expand_queue_down.insert(cell);
 			}
 
 			while (!expand_queue_up.empty() || !expand_queue_down.empty())
@@ -1153,7 +1175,7 @@ struct Abc9Pass : public Pass {
 
 					for (auto bit : cell_to_bit_up[cell])
 					for (auto c : bit_to_cell_up[bit])
-						if (unassigned_cells.count(c) && !c->type.in("$__ABC9_FF_", "$__ABC9_ASYNC_")) {
+						if (unassigned_cells.count(c)) {
 							unassigned_cells.erase(c);
 							next_expand_queue_up.insert(c);
 							assigned_cells[key].insert(c->name);