caiyu/nextpnr
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tmg: Fix argument order in run method

Browse Source 
Router 2 expects "update_route_delays" to be the first argument to `tmg.run`.
This commit is contained in:
rowanG077 2023-09-25 11:35:25 +02:00 committed by myrtle
parent 8e84006ee7
commit 38d2a4b844
2 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
common/kernel/timing.cc
View File

@ -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)

4
common/kernel/timing.h
View File

@ -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