cleaned up names of routines which check whether floorplan regions are overfull

Author: sfkhalid

vpr/src/pack/cluster_util.cpp

@@ -250,10 +250,6 @@ void free_clustering_data(const t_packer_opts& packer_opts,
     free(primitives_list);
 }
 
-void check_floorplan_regions(bool& floorplan_regions_overfull) {
-    floorplan_regions_overfull = check_clusters_floorplan_feasibility();
-}
-
 //check the clustering and output it
 void check_and_output_clustering(const t_packer_opts& packer_opts,
                                  const std::unordered_set<AtomNetId>& is_clock,
@@ -272,7 +268,8 @@ void check_and_output_clustering(const t_packer_opts& packer_opts,
 
     output_clustering(intra_lb_routing, packer_opts.global_clocks, is_clock, arch->architecture_id, packer_opts.output_file.c_str(), false);
 
-    check_floorplan_regions(floorplan_regions_overfull);
+    //check_floorplan_regions(floorplan_regions_overfull);
+    floorplan_regions_overfull = floorplan_constraints_regions_overfull();
 
     VTR_ASSERT(cluster_ctx.clb_nlist.blocks().size() == intra_lb_routing.size());
 }

vpr/src/pack/cluster_util.h

@@ -84,8 +84,6 @@ void free_clustering_data(const t_packer_opts& packer_opts,
                           t_molecule_link* memory_pool,
                           t_pb_graph_node** primitives_list);
 
-void check_floorplan_regions(bool& floorplan_regions_overfull);
-
 //check clustering legality and output it
 void check_and_output_clustering(const t_packer_opts& packer_opts,
                                  const std::unordered_set<AtomNetId>& is_clock,

vpr/src/pack/constraints_report.cpp

@@ -1,13 +1,6 @@
-/*
- * constraints_report.cpp
- *
- *  Created on: Dec. 7, 2021
- *      Author: khalid88
- */
-
 #include "constraints_report.h"
 
-bool check_clusters_floorplan_feasibility() {
+bool floorplan_constraints_regions_overfull() {
     GridTileLookup grid_tiles;
 
     auto& cluster_ctx = g_vpr_ctx.clustering();

vpr/src/pack/constraints_report.h

@@ -1,8 +1,5 @@
-/*
- * constraints_report.h
- *
- *  Created on: Dec. 7, 2021
- *      Author: khalid88
+/* Perform a check at the end of each packing iteration to see whether any
+ * floorplan regions have been packed with too many clusters.
  */
 
 #ifndef VPR_SRC_PACK_CONSTRAINTS_REPORT_H_
@@ -19,6 +16,6 @@
  * If the region is overfull, a message is printed saying which
  * region is overfull, and by how many clusters.
  */
-bool check_clusters_floorplan_feasibility();
+bool floorplan_constraints_regions_overfull();
 
 #endif /* VPR_SRC_PACK_CONSTRAINTS_REPORT_H_ */