Skip to content

Commit 1afe1c5

Browse files
committed
Don't compare expected node count with the rr_nodes count in the presence of dedicated clock networks
1 parent b70e728 commit 1afe1c5

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

vpr/src/route/rr_graph.cpp

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1342,9 +1342,10 @@ static void build_rr_graph(const t_graph_type graph_type,
13421342
is_flat);
13431343

13441344
// Verify no incremental node allocation.
1345-
/* AA: Note that in the case of dedicated networks, we are currently underestimating the additional node count due to the clock networks.
1346-
* Thus, this below error is logged; it's not actually an error, the node estimation needs to get fixed for dedicated clock networks. */
1347-
if (rr_graph.num_nodes() > expected_node_count) {
1345+
// AA: Note that in the case of dedicated networks, we are currently underestimating the additional node count due to the clock networks.
1346+
/* For now, the node count comparison is being skipped in the presence of clock networks.
1347+
* TODO: The node estimation needs to be fixed for dedicated clock networks. */
1348+
if (rr_graph.num_nodes() > expected_node_count && clock_modeling != DEDICATED_NETWORK) {
13481349
VTR_LOG_ERROR("Expected no more than %zu nodes, have %zu nodes\n",
13491350
expected_node_count, rr_graph.num_nodes());
13501351
}
@@ -1483,8 +1484,6 @@ static void build_intra_cluster_rr_graph(const t_graph_type graph_type,
14831484
is_flat,
14841485
load_rr_graph);
14851486

1486-
/* AA: Note that in the case of dedicated networks, we are currently underestimating the additional node count due to the clock networks.
1487-
* Thus this below error is logged; it's not actually an error, the node estimation needs to get fixed for dedicated clock networks. */
14881487
if (rr_graph.num_nodes() > expected_node_count) {
14891488
VTR_LOG_ERROR("Expected no more than %zu nodes, have %zu nodes\n",
14901489
expected_node_count, rr_graph.num_nodes());

0 commit comments

Comments
 (0)