From 0f947ee693b9afe4787efe9c145de7e533fa44de Mon Sep 17 00:00:00 2001
From: Rowan Goemans <goemansrowan@gmail.com>
Date: Mon, 12 Jun 2023 10:25:01 +0200
Subject: [PATCH] Timing: Fix combinational paths through all ports (#1175)

Fixes https://github.com/YosysHQ/nextpnr/issues/1174
---
 common/kernel/timing.cc | 48 +++++++++++++++++++----------------------
 1 file changed, 22 insertions(+), 26 deletions(-)

diff --git a/common/kernel/timing.cc b/common/kernel/timing.cc
index 7888e34a..df159c79 100644
--- a/common/kernel/timing.cc
+++ b/common/kernel/timing.cc
@@ -113,23 +113,21 @@ void TimingAnalyser::get_cell_delays()
                                               info.edge);
                 }
             }
-            // Combinational delays through cell
-            else if (cls == TMG_COMB_INPUT) {
-                for (auto &other_port : ci->ports) {
-                    auto &op = other_port.second;
-                    // ignore dangling ports and non-outputs
-                    if (op.net == nullptr || op.type != PORT_OUT)
-                        continue;
-                    DelayQuad delay;
-                    bool is_path = ctx->getCellDelay(ci, name, other_port.first, delay);
-                    if (is_path)
-                        pd.cell_arcs.emplace_back(CellArc::COMBINATIONAL, other_port.first, delay);
-                }
-            }
             // asynchronous endpoint
             else if (cls == TMG_ENDPOINT) {
                 pd.cell_arcs.emplace_back(CellArc::ENDPOINT, async_clk_key.key.clock, DelayQuad {});
             }
+            // Combinational delays through cell
+            for (auto &other_port : ci->ports) {
+                auto &op = other_port.second;
+                // ignore dangling ports and non-outputs
+                if (op.net == nullptr || op.type != PORT_OUT)
+                    continue;
+                DelayQuad delay;
+                bool is_path = ctx->getCellDelay(ci, name, other_port.first, delay);
+                if (is_path)
+                    pd.cell_arcs.emplace_back(CellArc::COMBINATIONAL, other_port.first, delay);
+            }
         } else if (pi.type == PORT_OUT) {
             // Output ports might have clk-to-q relationships
             if (cls == TMG_REGISTER_OUTPUT) {
@@ -140,23 +138,21 @@ void TimingAnalyser::get_cell_delays()
                     pd.cell_arcs.emplace_back(CellArc::CLK_TO_Q, info.clock_port, info.clockToQ, info.edge);
                 }
             }
-            // Combinational delays through cell
-            else if (cls == TMG_COMB_OUTPUT) {
-                for (auto &other_port : ci->ports) {
-                    auto &op = other_port.second;
-                    // ignore dangling ports and non-inputs
-                    if (op.net == nullptr || op.type != PORT_IN)
-                        continue;
-                    DelayQuad delay;
-                    bool is_path = ctx->getCellDelay(ci, other_port.first, name, delay);
-                    if (is_path)
-                        pd.cell_arcs.emplace_back(CellArc::COMBINATIONAL, other_port.first, delay);
-                }
-            }
             // Asynchronous startpoint
             else if (cls == TMG_STARTPOINT) {
                 pd.cell_arcs.emplace_back(CellArc::STARTPOINT, async_clk_key.key.clock, DelayQuad {});
             }
+            // Combinational delays through cell
+            for (auto &other_port : ci->ports) {
+                auto &op = other_port.second;
+                // ignore dangling ports and non-inputs
+                if (op.net == nullptr || op.type != PORT_IN)
+                    continue;
+                DelayQuad delay;
+                bool is_path = ctx->getCellDelay(ci, other_port.first, name, delay);
+                if (is_path)
+                    pd.cell_arcs.emplace_back(CellArc::COMBINATIONAL, other_port.first, delay);
+            }
         }
     }
 }