address PR comments

Author: soheilshahrouz

vpr/src/base/vpr_context.h

@@ -345,7 +345,7 @@ struct ClusteringHelperContext : public Context {
     t_ext_pin_util_targets target_external_pin_util;
 
     // During clustering, a block is related to un-clustered primitives with nets.
-    // This relation has three types: low fanout, high fanout, and trasitive
+    // This relation has three types: low fanout, high fanout, and transitive
     // high_fanout_thresholds stores the threshold for nets to a block type to be considered high fanout
     t_pack_high_fanout_thresholds high_fanout_thresholds;
 
@@ -386,9 +386,10 @@ struct PlacementContext : public Context {
     ///@brief The pl_macros array stores all the placement macros (usually carry chains).
     std::vector<t_pl_macro> pl_macros;
 
-    ///@brief Stores ClusterBlockId of all movable clustered blocks.
+    ///@brief Stores ClusterBlockId of all movable clustered blocks (blocks that are not locked down to a single location)
     std::vector<ClusterBlockId> movable_blocks;
 
+    ///@brief Stores ClusterBlockId of all movable clustered of each block type
     std::unordered_map<int, std::vector<ClusterBlockId>> movable_blocks_per_type;
 
     /**

vpr/src/place/initial_placement.cpp

@@ -225,7 +225,7 @@ static void check_initial_placement_legality();
 /**
  * @brief Fills movable_blocks in global PlacementContext
  */
-static void fill_movable_blocks();
+static void alloc_and_load_movable_blocks();
 
 static void check_initial_placement_legality() {
     auto& cluster_ctx = g_vpr_ctx.clustering();
@@ -1154,7 +1154,7 @@ bool place_one_block(const ClusterBlockId& blk_id,
     return placed_macro;
 }
 
-static void fill_movable_blocks() {
+static void alloc_and_load_movable_blocks() {
     auto& place_ctx = g_vpr_ctx.mutable_placement();
     auto& cluster_ctx = g_vpr_ctx.clustering();
     auto& device_ctx = g_vpr_ctx.device();
@@ -1225,7 +1225,7 @@ void initial_placement(const t_placer_opts& placer_opts,
     //Place all blocks
     place_all_blocks(placer_opts, block_scores, placer_opts.pad_loc_type, constraints_file);
 
-    fill_movable_blocks();
+    alloc_and_load_movable_blocks();
 
     // ensure all blocks are placed and that NoC routing has no cycles
     check_initial_placement_legality();