From 38d2a4b8444bd32e46cf135d1defe75b85280c46 Mon Sep 17 00:00:00 2001 From: rowanG077 Date: Mon, 25 Sep 2023 11:35:25 +0200 Subject: [PATCH] tmg: Fix argument order in run method Router 2 expects "update_route_delays" to be the first argument to `tmg.run`. --- common/kernel/timing.cc | 6 +++--- common/kernel/timing.h | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/common/kernel/timing.cc b/common/kernel/timing.cc index 36813265..7b66c52b 100644 --- a/common/kernel/timing.cc +++ b/common/kernel/timing.cc @@ -45,11 +45,11 @@ void TimingAnalyser::setup(bool update_net_timings, bool update_histogram, bool topo_sort(); setup_port_domains(); identify_related_domains(); - run(update_net_timings, update_histogram, update_crit_paths, true); + run(true, update_net_timings, update_histogram, update_crit_paths); } -void TimingAnalyser::run(bool update_net_timings, bool update_histogram, bool update_crit_paths, - bool update_route_delays) +void TimingAnalyser::run(bool update_route_delays, bool update_net_timings, bool update_histogram, + bool update_crit_paths) { reset_times(); if (update_route_delays) diff --git a/common/kernel/timing.h b/common/kernel/timing.h index 2610c618..3dabe4f1 100644 --- a/common/kernel/timing.h +++ b/common/kernel/timing.h @@ -77,8 +77,8 @@ struct TimingAnalyser TimingAnalyser(Context *ctx); void setup(bool update_net_timings = false, bool update_histogram = false, bool update_crit_paths = false); - void run(bool update_net_timings = false, bool update_histogram = false, bool update_crit_paths = false, - bool update_route_delays = true); + void run(bool update_route_delays = true, bool update_net_timings = false, bool update_histogram = false, + bool update_crit_paths = false); // This is used when routers etc are not actually binding detailed routing (due to congestion or an abstracted // model), but want to re-run STA with their own calculated delays