[vpr] apply PR comments

Author: amin1377

vpr/src/place/centroid_move_generator.cpp

@@ -38,7 +38,7 @@ e_create_move CentroidMoveGenerator::propose_move(t_pl_blocks_to_be_moved& block
     /* Calculate the centroid location*/
     calculate_centroid_loc(b_from, false, centroid, nullptr);
 
-    // Centroid location is not necessarily a valid location, and the downstream location expect a valid
+    // Centroid location is not necessarily a valid location, and the downstream location expects a valid
     // layer for the centroid location. So if the layer is not valid, we set it to the same layer as from loc.
     centroid.layer = (centroid.layer < 0) ? from.layer : centroid.layer;
     /* Find a location near the weighted centroid_loc */

vpr/src/place/place.cpp

@@ -2928,8 +2928,8 @@ static void get_bb_from_scratch(ClusterNetId net_id,
     coords.ymax = ymax;
     coords.layer_min = layer_min;
     coords.layer_max = layer_max;
-    VTR_ASSERT(layer_min >= 0 && layer_min < device_ctx.grid.get_num_layers());
-    VTR_ASSERT(layer_max >= 0 && layer_max < device_ctx.grid.get_num_layers());
+    VTR_ASSERT_DEBUG(layer_min >= 0 && layer_min < device_ctx.grid.get_num_layers());
+    VTR_ASSERT_DEBUG(layer_max >= 0 && layer_max < device_ctx.grid.get_num_layers());
 
 
     num_on_edges.xmin = xmin_edge;

vpr/src/place/placer_context.h

@@ -111,6 +111,8 @@ struct PlacerMoveContext : public Context {
 
     // Scratch vectors that are used by different directed moves for temporary calculations (allocated here to save runtime)
     // These vectors will grow up with the net size as it is mostly used to save coords of the net pins or net bb edges
+    // Given that placement moves involve operations on each coordinate independently, we chose to 
+    // utilize a Struct of Arrays (SoA) rather than an Array of Struct (AoS).
     std::vector<int> X_coord;
     std::vector<int> Y_coord;
     std::vector<int> layer_coord;