From: Eddie Hung <eddie@fpgeh.com>
Date: Wed, 28 Aug 2019 17:11:09 +0000 (-0700)
Subject: More cleanup
X-Git-Tag: working-ls180~1085^2~5
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=86b538bd02cb3e4c7e14be2c6d2210eb39463b0f;p=yosys.git

More cleanup
---

diff --git a/passes/pmgen/xilinx_srl.cc b/passes/pmgen/xilinx_srl.cc
index b3bab6021..f66582025 100644
--- a/passes/pmgen/xilinx_srl.cc
+++ b/passes/pmgen/xilinx_srl.cc
@@ -36,8 +36,6 @@ void run_fixed(xilinx_srl_pm &pm)
 	auto &ud = pm.ud_fixed;
 	log("Found fixed chain of length %d (%s):\n", GetSize(ud.longest_chain), log_id(st.first->type));
 
-	auto first_cell = ud.longest_chain.back();
-
 	SigSpec initval;
 	for (auto cell : ud.longest_chain) {
 		log_debug("    %s\n", log_id(cell));
@@ -61,10 +59,10 @@ void run_fixed(xilinx_srl_pm &pm)
 		}
 		else
 			log_abort();
-		if (cell != first_cell)
-			pm.autoremove(cell);
+		pm.autoremove(cell);
 	}
 
+	auto first_cell = ud.longest_chain.back();
 	auto last_cell = ud.longest_chain.front();
 	Cell *c = pm.module->addCell(NEW_ID, ID($__XILINX_SHREG_));
 	pm.module->swap_names(c, first_cell);
@@ -117,9 +115,6 @@ void run_variable(xilinx_srl_pm &pm)
 
 	log("Found variable chain of length %d (%s):\n", GetSize(ud.chain), log_id(st.first->type));
 
-	auto first_cell = ud.chain.back().first;
-	auto first_slice = ud.chain.back().second;
-
 	SigSpec initval;
 	for (const auto &i : ud.chain) {
 		auto cell = i.first;
@@ -139,11 +134,13 @@ void run_variable(xilinx_srl_pm &pm)
 		}
 		else
 			log_abort();
-		if (cell != first_cell)
-			cell->connections_.at(ID(Q))[slice] = pm.module->addWire(NEW_ID);
+		cell->connections_.at(ID(Q))[slice] = pm.module->addWire(NEW_ID);
 	}
 	pm.autoremove(st.shiftx);
 
+	auto first_cell = ud.chain.back().first;
+	auto first_slice = ud.chain.back().second;
+
 	Cell *c = pm.module->addCell(NEW_ID, ID($__XILINX_SHREG_));
 	pm.module->swap_names(c, first_cell);