Skip to content

Commit dabeec3

Browse files
committed
make format
1 parent 58ed574 commit dabeec3

11 files changed

+170
-179
lines changed

vpr/src/base/vpr_api.cpp

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -922,19 +922,19 @@ RouteStatus vpr_route_fixed_W(const Netlist<>& net_list,
922922
}
923923
bool status = false;
924924
status = route(net_list,
925-
fixed_channel_width,
926-
vpr_setup.RouterOpts,
927-
vpr_setup.AnalysisOpts,
928-
&vpr_setup.RoutingArch,
929-
vpr_setup.Segments,
930-
net_delay,
931-
timing_info,
932-
delay_calc,
933-
arch.Chans,
934-
arch.Directs,
935-
arch.num_directs,
936-
ScreenUpdatePriority::MAJOR,
937-
is_flat);
925+
fixed_channel_width,
926+
vpr_setup.RouterOpts,
927+
vpr_setup.AnalysisOpts,
928+
&vpr_setup.RoutingArch,
929+
vpr_setup.Segments,
930+
net_delay,
931+
timing_info,
932+
delay_calc,
933+
arch.Chans,
934+
arch.Directs,
935+
arch.num_directs,
936+
ScreenUpdatePriority::MAJOR,
937+
is_flat);
938938

939939
return RouteStatus(status, fixed_channel_width);
940940
}

vpr/src/route/netlist_routers.h

Lines changed: 46 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -37,16 +37,16 @@ struct RouteIterResults {
3737

3838
/** Route a given netlist. Takes a big context and passes it around to net & sink routing fns. */
3939
class NetlistRouter {
40-
public:
40+
public:
4141
virtual ~NetlistRouter() {}
4242
virtual RouteIterResults route_netlist(int itry, float pres_fac, float worst_neg_slack) = 0;
4343
virtual void set_rcv_enabled(bool x) = 0;
4444
virtual void set_timing_info(std::shared_ptr<SetupHoldTimingInfo> timing_info) = 0;
4545
};
4646

4747
template<typename HeapType>
48-
class SerialNetlistRouter: public NetlistRouter {
49-
public:
48+
class SerialNetlistRouter : public NetlistRouter {
49+
public:
5050
SerialNetlistRouter(
5151
const Netlist<>& net_list,
5252
const RouterLookahead* router_lookahead,
@@ -59,27 +59,27 @@ class SerialNetlistRouter: public NetlistRouter {
5959
route_budgets& budgeting_inf,
6060
const RoutingPredictor& routing_predictor,
6161
const vtr::vector<ParentNetId, std::vector<std::unordered_map<RRNodeId, int>>>& choking_spots,
62-
bool is_flat
63-
):
64-
_router(_make_router(router_lookahead, is_flat)),
65-
_net_list(net_list),
66-
_router_opts(router_opts),
67-
_connections_inf(connections_inf),
68-
_net_delay(net_delay),
69-
_netlist_pin_lookup(netlist_pin_lookup),
70-
_timing_info(timing_info),
71-
_pin_timing_invalidator(pin_timing_invalidator),
72-
_budgeting_inf(budgeting_inf),
73-
_routing_predictor(routing_predictor),
74-
_choking_spots(choking_spots),
75-
_is_flat(is_flat){}
62+
bool is_flat)
63+
: _router(_make_router(router_lookahead, is_flat))
64+
, _net_list(net_list)
65+
, _router_opts(router_opts)
66+
, _connections_inf(connections_inf)
67+
, _net_delay(net_delay)
68+
, _netlist_pin_lookup(netlist_pin_lookup)
69+
, _timing_info(timing_info)
70+
, _pin_timing_invalidator(pin_timing_invalidator)
71+
, _budgeting_inf(budgeting_inf)
72+
, _routing_predictor(routing_predictor)
73+
, _choking_spots(choking_spots)
74+
, _is_flat(is_flat) {}
7675
~SerialNetlistRouter() {}
7776

7877
RouteIterResults route_netlist(int itry, float pres_fac, float worst_neg_slack);
7978
void set_rcv_enabled(bool x);
8079
void set_timing_info(std::shared_ptr<SetupHoldTimingInfo> timing_info);
81-
private:
82-
ConnectionRouter<HeapType> _make_router(const RouterLookahead* router_lookahead, bool is_flat){
80+
81+
private:
82+
ConnectionRouter<HeapType> _make_router(const RouterLookahead* router_lookahead, bool is_flat) {
8383
auto& device_ctx = g_vpr_ctx.device();
8484
auto& route_ctx = g_vpr_ctx.mutable_routing();
8585

@@ -109,8 +109,8 @@ class SerialNetlistRouter: public NetlistRouter {
109109
};
110110

111111
template<typename HeapType>
112-
class ParallelNetlistRouter: public NetlistRouter {
113-
public:
112+
class ParallelNetlistRouter : public NetlistRouter {
113+
public:
114114
ParallelNetlistRouter(
115115
const Netlist<>& net_list,
116116
const RouterLookahead* router_lookahead,
@@ -123,28 +123,28 @@ class ParallelNetlistRouter: public NetlistRouter {
123123
route_budgets& budgeting_inf,
124124
const RoutingPredictor& routing_predictor,
125125
const vtr::vector<ParentNetId, std::vector<std::unordered_map<RRNodeId, int>>>& choking_spots,
126-
bool is_flat
127-
):
128-
_routers_th(_make_router(router_lookahead, is_flat)),
129-
_net_list(net_list),
130-
_router_opts(router_opts),
131-
_connections_inf(connections_inf),
132-
_net_delay(net_delay),
133-
_netlist_pin_lookup(netlist_pin_lookup),
134-
_timing_info(timing_info),
135-
_pin_timing_invalidator(pin_timing_invalidator),
136-
_budgeting_inf(budgeting_inf),
137-
_routing_predictor(routing_predictor),
138-
_choking_spots(choking_spots),
139-
_is_flat(is_flat){}
126+
bool is_flat)
127+
: _routers_th(_make_router(router_lookahead, is_flat))
128+
, _net_list(net_list)
129+
, _router_opts(router_opts)
130+
, _connections_inf(connections_inf)
131+
, _net_delay(net_delay)
132+
, _netlist_pin_lookup(netlist_pin_lookup)
133+
, _timing_info(timing_info)
134+
, _pin_timing_invalidator(pin_timing_invalidator)
135+
, _budgeting_inf(budgeting_inf)
136+
, _routing_predictor(routing_predictor)
137+
, _choking_spots(choking_spots)
138+
, _is_flat(is_flat) {}
140139
~ParallelNetlistRouter() {}
141140

142141
RouteIterResults route_netlist(int itry, float pres_fac, float worst_neg_slack);
143142
void set_rcv_enabled(bool x);
144143
void set_timing_info(std::shared_ptr<SetupHoldTimingInfo> timing_info);
145-
private:
144+
145+
private:
146146
void route_partition_tree_node(tbb::task_group& g, PartitionTreeNode& node, int itry, float pres_fac, float worst_neg_slack);
147-
ConnectionRouter<HeapType> _make_router(const RouterLookahead* router_lookahead, bool is_flat){
147+
ConnectionRouter<HeapType> _make_router(const RouterLookahead* router_lookahead, bool is_flat) {
148148
auto& device_ctx = g_vpr_ctx.device();
149149
auto& route_ctx = g_vpr_ctx.mutable_routing();
150150

@@ -187,8 +187,8 @@ inline std::unique_ptr<NetlistRouter> make_netlist_router_with_heap(
187187
route_budgets& budgeting_inf,
188188
const RoutingPredictor& routing_predictor,
189189
const vtr::vector<ParentNetId, std::vector<std::unordered_map<RRNodeId, int>>>& choking_spots,
190-
bool is_flat){
191-
if(router_opts.router_algorithm == e_router_algorithm::TIMING_DRIVEN){
190+
bool is_flat) {
191+
if (router_opts.router_algorithm == e_router_algorithm::TIMING_DRIVEN) {
192192
return std::make_unique<SerialNetlistRouter<HeapType>>(
193193
net_list,
194194
router_lookahead,
@@ -202,7 +202,7 @@ inline std::unique_ptr<NetlistRouter> make_netlist_router_with_heap(
202202
routing_predictor,
203203
choking_spots,
204204
is_flat);
205-
}else if(router_opts.router_algorithm == e_router_algorithm::PARALLEL){
205+
} else if (router_opts.router_algorithm == e_router_algorithm::PARALLEL) {
206206
#ifdef VPR_USE_TBB
207207
return std::make_unique<ParallelNetlistRouter<HeapType>>(
208208
net_list,
@@ -220,7 +220,7 @@ inline std::unique_ptr<NetlistRouter> make_netlist_router_with_heap(
220220
#else
221221
VPR_FATAL_ERROR(VPR_ERROR_ROUTE, "VPR isn't compiled with TBB support required for parallel routing");
222222
#endif
223-
}else{
223+
} else {
224224
VPR_FATAL_ERROR(VPR_ERROR_ROUTE, "Unknown router algorithm %d", router_opts.router_algorithm);
225225
}
226226
}
@@ -239,7 +239,7 @@ inline std::unique_ptr<NetlistRouter> make_netlist_router(
239239
const RoutingPredictor& routing_predictor,
240240
const vtr::vector<ParentNetId, std::vector<std::unordered_map<RRNodeId, int>>>& choking_spots,
241241
bool is_flat) {
242-
if(router_opts.router_heap == e_heap_type::BINARY_HEAP){
242+
if (router_opts.router_heap == e_heap_type::BINARY_HEAP) {
243243
return make_netlist_router_with_heap<BinaryHeap>(
244244
net_list,
245245
router_lookahead,
@@ -252,9 +252,8 @@ inline std::unique_ptr<NetlistRouter> make_netlist_router(
252252
budgeting_inf,
253253
routing_predictor,
254254
choking_spots,
255-
is_flat
256-
);
257-
}else if(router_opts.router_heap == e_heap_type::BUCKET_HEAP_APPROXIMATION){
255+
is_flat);
256+
} else if (router_opts.router_heap == e_heap_type::BUCKET_HEAP_APPROXIMATION) {
258257
return make_netlist_router_with_heap<Bucket>(
259258
net_list,
260259
router_lookahead,
@@ -267,10 +266,8 @@ inline std::unique_ptr<NetlistRouter> make_netlist_router(
267266
budgeting_inf,
268267
routing_predictor,
269268
choking_spots,
270-
is_flat
271-
);
272-
}else{
269+
is_flat);
270+
} else {
273271
VPR_FATAL_ERROR(VPR_ERROR_ROUTE, "Unknown heap type %d", router_opts.router_heap);
274272
}
275273
}
276-

vpr/src/route/parallel_netlist_router.cpp

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55
/** A parallel netlist router. Builds a PartitionTree from the netlist depending on bounding boxes
66
* and goes over it with a tbb::task_group. Is not expected to use more than 2 effective threads on avg. */
77
template<typename HeapType>
8-
RouteIterResults ParallelNetlistRouter<HeapType>::route_netlist(int itry, float pres_fac, float worst_neg_slack){
9-
for(auto& results: _results_th){
8+
RouteIterResults ParallelNetlistRouter<HeapType>::route_netlist(int itry, float pres_fac, float worst_neg_slack) {
9+
for (auto& results : _results_th) {
1010
results = RouteIterResults();
1111
}
1212

@@ -26,7 +26,7 @@ RouteIterResults ParallelNetlistRouter<HeapType>::route_netlist(int itry, float
2626
}
2727

2828
template<typename HeapType>
29-
void ParallelNetlistRouter<HeapType>::route_partition_tree_node(tbb::task_group& g, PartitionTreeNode& node, int itry, float pres_fac, float worst_neg_slack){
29+
void ParallelNetlistRouter<HeapType>::route_partition_tree_node(tbb::task_group& g, PartitionTreeNode& node, int itry, float pres_fac, float worst_neg_slack) {
3030
auto& route_ctx = g_vpr_ctx.mutable_routing();
3131

3232
/* Sort so net with most sinks is routed first. */
@@ -83,18 +83,17 @@ void ParallelNetlistRouter<HeapType>::route_partition_tree_node(tbb::task_group&
8383
}
8484

8585
template<typename HeapType>
86-
void ParallelNetlistRouter<HeapType>::set_rcv_enabled(bool x){
87-
for(auto& router: _routers_th){
86+
void ParallelNetlistRouter<HeapType>::set_rcv_enabled(bool x) {
87+
for (auto& router : _routers_th) {
8888
router.set_rcv_enabled(x);
8989
}
9090
}
9191

9292
template<typename HeapType>
93-
void ParallelNetlistRouter<HeapType>::set_timing_info(std::shared_ptr<SetupHoldTimingInfo> timing_info){
93+
void ParallelNetlistRouter<HeapType>::set_timing_info(std::shared_ptr<SetupHoldTimingInfo> timing_info) {
9494
_timing_info = timing_info;
9595
}
9696

97-
9897
/** Explicit instantiation so that the compiler knows it should emit code for these cases */
9998
template class ParallelNetlistRouter<BinaryHeap>;
10099
template class ParallelNetlistRouter<Bucket>;

vpr/src/route/route.cpp

Lines changed: 11 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -187,15 +187,15 @@ bool route(const Netlist<>& net_list,
187187
int legal_convergence_count = 0;
188188

189189
/* Get initial criticalities from the lookahead */
190-
if(router_opts.with_timing_analysis && router_opts.initial_timing == e_router_initial_timing::LOOKAHEAD){
190+
if (router_opts.with_timing_analysis && router_opts.initial_timing == e_router_initial_timing::LOOKAHEAD) {
191191
vtr::ScopedStartFinishTimer init_timing_timer("Initializing router criticalities");
192192
//Estimate initial connection delays from the router lookahead
193193
init_net_delay_from_lookahead(*router_lookahead,
194-
net_list,
195-
route_ctx.net_rr_terminals,
196-
net_delay,
197-
device_ctx.rr_graph,
198-
is_flat);
194+
net_list,
195+
route_ctx.net_rr_terminals,
196+
net_delay,
197+
device_ctx.rr_graph,
198+
is_flat);
199199

200200
//Run STA to get estimated criticalities
201201
timing_info->update();
@@ -225,8 +225,7 @@ bool route(const Netlist<>& net_list,
225225
budgeting_inf,
226226
routing_predictor,
227227
choking_spots,
228-
is_flat
229-
);
228+
is_flat);
230229

231230
RouterStats router_stats;
232231
float prev_iter_cumm_time = 0;
@@ -265,15 +264,15 @@ bool route(const Netlist<>& net_list,
265264
}
266265

267266
/* Initial criticalities: set to 1 on the first iter if the user asked for it */
268-
if(router_opts.initial_timing == e_router_initial_timing::ALL_CRITICAL && itry == 1)
267+
if (router_opts.initial_timing == e_router_initial_timing::ALL_CRITICAL && itry == 1)
269268
netlist_router->set_timing_info(make_constant_timing_info(1));
270269
else
271270
netlist_router->set_timing_info(timing_info);
272271

273272
/* Route each net */
274273
RouteIterResults iter_results = netlist_router->route_netlist(itry, pres_fac, worst_negative_slack);
275274

276-
if(!iter_results.is_routable){ /* Disconnected RRG */
275+
if (!iter_results.is_routable) { /* Disconnected RRG */
277276
return false;
278277
}
279278

@@ -351,7 +350,7 @@ bool route(const Netlist<>& net_list,
351350
success = true;
352351

353352
//Update best metrics
354-
if(router_opts.with_timing_analysis){
353+
if (router_opts.with_timing_analysis) {
355354
check_net_delays(net_list, net_delay);
356355
best_routing_metrics.sTNS = timing_info->setup_total_negative_slack();
357356
best_routing_metrics.sWNS = timing_info->setup_worst_negative_slack();
@@ -601,7 +600,7 @@ bool route(const Netlist<>& net_list,
601600
#endif
602601
}
603602

604-
if(router_opts.with_timing_analysis){
603+
if (router_opts.with_timing_analysis) {
605604
VTR_LOG("Final Net Connection Criticality Histogram:\n");
606605
print_router_criticality_histogram(net_list, *timing_info, netlist_pin_lookup, is_flat);
607606
}

vpr/src/route/route_common.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -158,8 +158,8 @@ void try_graph(int width_fac,
158158
}
159159

160160
/** This routine checks to see if this is a resource-feasible routing.
161-
* That is, are all rr_node capacity limitations respected? It assumes
162-
* that the occupancy arrays are up to date when it is called. */
161+
* That is, are all rr_node capacity limitations respected? It assumes
162+
* that the occupancy arrays are up to date when it is called. */
163163
bool feasible_routing() {
164164
auto& device_ctx = g_vpr_ctx.device();
165165
const auto& rr_graph = device_ctx.rr_graph;

vpr/src/route/route_net.cpp

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -206,12 +206,12 @@ bool early_exit_heuristic(const t_router_opts& router_opts, const WirelengthInfo
206206
}
207207

208208
float get_net_pin_criticality(const SetupHoldTimingInfo* timing_info,
209-
const ClusteredPinAtomPinsLookup& netlist_pin_lookup,
210-
float max_criticality,
211-
float criticality_exp,
212-
ParentNetId net_id,
213-
ParentPinId pin_id,
214-
bool is_flat) {
209+
const ClusteredPinAtomPinsLookup& netlist_pin_lookup,
210+
float max_criticality,
211+
float criticality_exp,
212+
ParentNetId net_id,
213+
ParentPinId pin_id,
214+
bool is_flat) {
215215
float pin_criticality = 0.0;
216216
const auto& route_ctx = g_vpr_ctx.routing();
217217

@@ -363,7 +363,6 @@ bool early_reconvergence_exit_heuristic(const t_router_opts& router_opts,
363363
return false; // Don't give up
364364
}
365365

366-
367366
// Initializes net_delay based on best-case delay estimates from the router lookahead
368367
void init_net_delay_from_lookahead(const RouterLookahead& router_lookahead,
369368
const Netlist<>& net_list,
@@ -399,10 +398,10 @@ void init_net_delay_from_lookahead(const RouterLookahead& router_lookahead,
399398
/** Goes through all the sinks of this net and copies their delay values from
400399
* the route_tree to the net_delay array. */
401400
void update_net_delays_from_route_tree(float* net_delay,
402-
const Netlist<>& net_list,
403-
ParentNetId inet,
404-
TimingInfo* timing_info,
405-
NetPinTimingInvalidator* pin_timing_invalidator) {
401+
const Netlist<>& net_list,
402+
ParentNetId inet,
403+
TimingInfo* timing_info,
404+
NetPinTimingInvalidator* pin_timing_invalidator) {
406405
auto& route_ctx = g_vpr_ctx.routing();
407406
const RouteTree& tree = route_ctx.route_trees[inet].value();
408407

0 commit comments

Comments
 (0)