add comments and address compilation warnings

Author: soheilshahrouz

vpr/src/place/net_cost_handler.cpp

@@ -583,7 +583,7 @@ void NetCostHandler::get_non_updatable_per_layer_bb_(ClusterNetId net_id, bool u
     vtr::NdMatrixProxy<int, 1> num_sink_layer = use_ts ? ts_layer_sink_pin_count_[size_t(net_id)] : move_ctx.num_sink_pin_layer[size_t(net_id)];
 
     const int num_layers = device_ctx.grid.get_num_layers();
-    VTR_ASSERT_DEBUG(bb_coord_new.size() == num_layers);
+    VTR_ASSERT_DEBUG(bb_coord_new.size() == (size_t)num_layers);
 
     ClusterBlockId bnum = cluster_ctx.clb_nlist.net_driver_block(net_id);
     t_pl_loc block_loc = block_locs[bnum].loc;
@@ -1388,8 +1388,8 @@ void NetCostHandler::get_layer_bb_from_scratch_(ClusterNetId net_id,
     auto& block_locs = placer_state_.block_locs();
 
     const int num_layers = device_ctx.grid.get_num_layers();
-    VTR_ASSERT_DEBUG(coords.size() == num_layers);
-    VTR_ASSERT_DEBUG(num_on_edges.size() == num_layers);
+    VTR_ASSERT_DEBUG(coords.size() == (size_t)num_layers);
+    VTR_ASSERT_DEBUG(num_on_edges.size() == (size_t)num_layers);
 
     ClusterBlockId bnum = cluster_ctx.clb_nlist.net_driver_block(net_id);
     t_pl_loc block_loc = block_locs[bnum].loc;

vpr/src/place/net_cost_handler.h

@@ -137,9 +137,9 @@ class NetCostHandler {
     std::function<double(e_cost_methods method)> comp_bb_cost_functor_;
     ///@brief Points to the proper method for updating the bounding box of a net.
     std::function<void(ClusterNetId net_id, t_physical_tile_loc pin_old_loc, t_physical_tile_loc pin_new_loc, bool is_driver)> update_bb_functor_;
-
+    ///@brief Points to the proper method for getting the bounding box cost of a net
     std::function<double(ClusterNetId)> get_net_bb_cost_functor_;
-
+    ///@brief Points to the proper method for getting the non-updatable bounding box of a net
     std::function<void(const ClusterNetId net)> get_non_updatable_bb_functor_;
 
     /**
@@ -268,10 +268,18 @@ class NetCostHandler {
                                 double& delta_timing_cost,
                                 bool is_src_moving);
 
-
-
+    /**
+     * @brief Updates the bounding box coordinates of a net in the placer state
+     * with coordinates stored in the `ts` bounding box coordinates container.
+     * @param net_id ID of the net whose bounding box coordinates it to be updated.
+     */
     void set_ts_bb_coord_(const ClusterNetId net_id);
 
+    /**
+     * @brief Updates the number of pins on each boundary of the bounding box for a net
+     * in the placer state with the number of pins stores in the 'ts' num_on_edges containers.
+     * @param net_id ID of the net whose number of pins on each BB edge is to be updated.
+     */
     void set_ts_edge_(const ClusterNetId net_id);
 
     /**