removed unused arguments

Author: soheilshahrouz

vpr/src/place/initial_noc_placement.cpp

@@ -240,7 +240,7 @@ static void noc_routers_anneal(const t_noc_opts& noc_opts) {
     }
 
     if (checkpoint.get_cost() < costs.cost) {
-        checkpoint.restore_checkpoint(noc_opts, costs);
+        checkpoint.restore_checkpoint(costs);
     }
 }
 

vpr/src/place/noc_place_checkpoint.cpp

@@ -32,7 +32,7 @@ void NoCPlacementCheckpoint::save_checkpoint(double cost) {
     cost_ = cost;
 }
 
-void NoCPlacementCheckpoint::restore_checkpoint(const t_noc_opts& noc_opts, t_placer_costs& costs) {
+void NoCPlacementCheckpoint::restore_checkpoint(t_placer_costs& costs) {
     const auto& noc_ctx = g_vpr_ctx.noc();
     const auto& device_ctx = g_vpr_ctx.device();
     auto& place_ctx = g_vpr_ctx.mutable_placement();
@@ -68,7 +68,7 @@ void NoCPlacementCheckpoint::restore_checkpoint(const t_noc_opts& noc_opts, t_pl
     }
 
     // Re-initialize routes and static variables that keep track of NoC-related costs
-    reinitialize_noc_routing(noc_opts, costs);
+    reinitialize_noc_routing(costs);
 }
 
 bool NoCPlacementCheckpoint::is_valid() const {

vpr/src/place/noc_place_checkpoint.h

@@ -43,10 +43,9 @@ class NoCPlacementCheckpoint {
     /**
      * @brief Loads the save checkpoint into global placement data structues.
      *
-     *  @param noc_opts: Contains weighting factors for different NoC cost terms
      *  @param costs: Used to load NoC related costs for the checkpoint
      */
-    void restore_checkpoint(const t_noc_opts& noc_opts, t_placer_costs& costs);
+    void restore_checkpoint(t_placer_costs& costs);
 
     /**
      * @brief Indicates whether the object is empty or it has already stored a

vpr/src/place/noc_place_utils.cpp

@@ -66,7 +66,7 @@ void initial_noc_routing(void) {
     return;
 }
 
-void reinitialize_noc_routing(const t_noc_opts& noc_opts, t_placer_costs& costs) {
+void reinitialize_noc_routing(t_placer_costs& costs) {
     // used to access NoC links and modify them
     auto& noc_ctx = g_vpr_ctx.mutable_noc();
 

vpr/src/place/noc_place_utils.h

@@ -70,10 +70,9 @@ void initial_noc_routing(void);
  * traffic flow routes, and static variable in noc_place_utils.cpp are no
  * longer valid and need to be re-initialized.
  *
- * @param noc_opts NoC-related options used to calculated NoC costs
  * @param costs Used to get aggregate bandwidth and latency costs.
  */
-void reinitialize_noc_routing(const t_noc_opts& noc_opts, t_placer_costs& costs);
+void reinitialize_noc_routing(t_placer_costs& costs);
 
 /**
  * @brief Goes through all the cluster blocks that were moved

vpr/src/place/place_checkpoint.cpp

@@ -63,7 +63,7 @@ void restore_best_placement(t_placement_checkpoint& placement_checkpoint,
          * and need to be re-computed from scratch.
          */
         if (noc_opts.noc) {
-            reinitialize_noc_routing(noc_opts, costs);
+            reinitialize_noc_routing(costs);
         }
 
         VTR_LOG("\nCheckpoint restored\n");