Added comments to routine that intersects range limit with floorplan constraints during place moves

Author: sfkhalid

vpr/src/pack/output_clustering.cpp

@@ -124,7 +124,6 @@ static void print_stats() {
     }
     VTR_LOG("Absorbed logical nets %d out of %d nets, %d nets not absorbed.\n",
             total_nets_absorbed, (int)atom_ctx.nlist.nets().size(), (int)atom_ctx.nlist.nets().size() - total_nets_absorbed);
-    VTR_LOG("Netlist num_blocks: %d \n", cluster_ctx.clb_nlist.blocks().size());
     free(num_clb_types);
     free(num_clb_inputs_used);
     free(num_clb_outputs_used);

vpr/src/place/move_utils.cpp

@@ -893,6 +893,13 @@ bool intersect_range_limit_with_floorplan_constraints(t_logical_block_type_ptr t
         regions = pr.get_partition_region();
     }
     Region intersect_reg;
+    /*
+     * If region size is greater than 1, the block is constrained to more than one rectangular region.
+     * In this case, we return true (i.e. the range limit intersects with
+     * the floorplan constraints) to simplify the problem. This simplification can be done because
+     * this routine is done for cpu time optimization, so we do not have to necessarily check each
+     * complicated case to get correct functionality during place moves.
+     */
     if (regions.size() == 1) {
         intersect_reg = intersection(regions[0], range_reg);
 

vpr/src/place/move_utils.h

@@ -179,6 +179,14 @@ bool find_compatible_compressed_loc_in_range(t_logical_block_type_ptr type, int
  *
  * Returns false if there is no intersection between the range limit and the floorplan constraint,
  * true otherwise.
+ *
+ *
+ * If region size of the block's floorplan constraints is greater than 1, the block is constrained to more than one rectangular region.
+ * In this case, we return true (i.e. the range limit intersects with
+ * the floorplan constraints) to simplify the problem. This simplification can be done because
+ * this routine is done for cpu time optimization, so we do not have to necessarily check each
+ * complicated case to get correct functionality during place moves.
+ *
  */
 bool intersect_range_limit_with_floorplan_constraints(t_logical_block_type_ptr type, ClusterBlockId b_from, int& min_cx, int& min_cy, int& max_cx, int& max_cy, int& delta_cx);