Skip to content

Commit c9afd34

Browse files
committed
[vpr][place] fix typos in comments
1 parent 2d7fbf4 commit c9afd34

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

vpr/src/place/net_cost_handler.cpp

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -240,10 +240,13 @@ static void get_non_updatable_layer_bb(ClusterNetId net_id,
240240

241241
/**
242242
* @brief Update the per-layer bounding box of "net_id" incrementally based on the old and new locations of a pin on that net
243-
* @details /* Updates the bounding box of a net by storing its coordinates in the bb_coord_new data structure and
243+
* @details Updates the bounding box of a net by storing its coordinates in the bb_coord_new data structure and
244244
* the number of blocks on each edge in the bb_edge_new data structure. This routine should only be called for
245245
* large nets, since it has some overhead relative to just doing a brute force bounding box calculation.
246-
* The bounding box coordinate and edge information for inet must be valid before this routine is called. Currently assumes channels on both sides of the CLBs forming the edges of the bounding box can be used. Essentially, I am assuming the pins always lie on the outside of the bounding box. The x and y coordinates are the pin's x and y coordinates. IO blocks are considered to be one cell in for simplicity.
246+
* The bounding box coordinate and edge information for inet must be valid before this routine is called.
247+
* Currently assumes channels on both sides of the CLBs forming the edges of the bounding box can be used.
248+
* Essentially, I am assuming the pins always lie on the outside of the bounding box. The x and y coordinates
249+
* are the pin's x and y coordinates. IO blocks are considered to be one cell in for simplicity.
247250
* @param bb_edge_new Number of blocks on the edges of the bounding box
248251
* @param bb_coord_new Coordinates of the bounding box
249252
* @param num_sink_pin_layer_new Number of sinks of the given net on each layer
@@ -441,7 +444,8 @@ static double recompute_bb_cost();
441444
static double wirelength_crossing_count(size_t fanout);
442445

443446
/**
444-
* @brief Calculates and returns the total bb (wirelength) cost change that would result from moving the blocks indicated in the blocks_affected data structure.
447+
* @brief Calculates and returns the total bb (wirelength) cost change that would result from moving the blocks
448+
* indicated in the blocks_affected data structure.
445449
* @param num_affected_nets Number of valid elements in ts_bb_coord_new
446450
* @param bb_delta_c Cost difference after and before moving the block
447451
*/
@@ -880,7 +884,8 @@ static void update_bb(ClusterNetId net_id,
880884
return;
881885
}
882886

883-
vtr::NdMatrixProxy<int, 1> curr_num_sink_pin_layer = (bb_updated_before[net_id] == NetUpdateState::NOT_UPDATED_YET) ? place_move_ctx.num_sink_pin_layer[size_t(net_id)] : num_sink_pin_layer_new;
887+
vtr::NdMatrixProxy<int, 1> curr_num_sink_pin_layer = (bb_updated_before[net_id] == NetUpdateState::NOT_UPDATED_YET) ?
888+
place_move_ctx.num_sink_pin_layer[size_t(net_id)] : num_sink_pin_layer_new;
884889

885890
if (bb_updated_before[net_id] == NetUpdateState::NOT_UPDATED_YET) {
886891
/* The net had NOT been updated before, could use the old values */
@@ -1150,7 +1155,8 @@ static void update_layer_bb(ClusterNetId net_id,
11501155
return;
11511156
}
11521157

1153-
const vtr::NdMatrixProxy<int, 1> curr_layer_pin_sink_count = (bb_updated_before[net_id] == NetUpdateState::NOT_UPDATED_YET) ? place_move_ctx.num_sink_pin_layer[size_t(net_id)] : bb_pin_sink_count_new;
1158+
const vtr::NdMatrixProxy<int, 1> curr_layer_pin_sink_count = (bb_updated_before[net_id] == NetUpdateState::NOT_UPDATED_YET) ?
1159+
place_move_ctx.num_sink_pin_layer[size_t(net_id)] : bb_pin_sink_count_new;
11541160

11551161
const std::vector<t_2D_bb>*curr_bb_edge, *curr_bb_coord;
11561162
if (bb_updated_before[net_id] == NetUpdateState::NOT_UPDATED_YET) {

0 commit comments

Comments
 (0)