Skip to content

Commit 69bd2f8

Browse files
committed
WIP: fixes path costs Tsw adjustment during lookahead computation
I believe the adjustment of the Tsw value was due to the use of connection boxes. This is not required anymore as connection boxes are not being used anymore Signed-off-by: Alessandro Comodi <[email protected]>
1 parent 31cf9d6 commit 69bd2f8

File tree

2 files changed

+1
-18
lines changed

2 files changed

+1
-18
lines changed

vpr/src/base/read_options.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1912,7 +1912,7 @@ argparse::ArgumentParser create_arg_parser(std::string prog_name, t_options& arg
19121912
" * classic: The classic VPR lookahead (may perform better on un-buffered routing\n"
19131913
" architectures)\n"
19141914
" * map: A more advanced lookahead which accounts for diverse wire type\n")
1915-
.default_value("map")
1915+
.default_value("classic")
19161916
.show_in(argparse::ShowIn::HELP_ONLY);
19171917

19181918
route_timing_grp.add_argument(args.router_max_convergence_count, "--router_max_convergence_count")

vpr/src/route/router_lookahead_map.cpp

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -591,21 +591,6 @@ static bool add_paths(int start_node_ind,
591591

592592
bool new_sample_found = false;
593593

594-
auto to_tile_type = device_ctx.grid[rr_graph.node_xlow(node)][rr_graph.node_ylow(node)].type;
595-
auto to_tile_index = to_tile_type->index;
596-
597-
auto to_ptc = rr_graph.node_ptc_num(node);
598-
599-
float site_pin_delay = std::numeric_limits<float>::infinity();
600-
if (f_chan_ipins_delays[to_tile_index].size() != 0) {
601-
for (const auto& kv : f_chan_ipins_delays[to_tile_index][to_ptc]) {
602-
const t_reachable_wire_inf& reachable_wire_inf = kv.second;
603-
604-
float this_delay = reachable_wire_inf.delay;
605-
site_pin_delay = std::min(this_delay, site_pin_delay);
606-
}
607-
}
608-
609594
// reconstruct the path
610595
std::vector<int> path;
611596
for (int i = paths[node_ind].parent; i != start_node_ind; i = paths[i].parent) {
@@ -614,8 +599,6 @@ static bool add_paths(int start_node_ind,
614599
}
615600
path.push_back(start_node_ind);
616601

617-
current.adjust_Tsw(-site_pin_delay);
618-
619602
// add each node along the path subtracting the incremental costs from the current costs
620603
Entry start_to_here(start_node_ind, UNDEFINED, nullptr);
621604
int parent = start_node_ind;

0 commit comments

Comments
 (0)