[Infra] Cleaned Up Header Files in Pack Folder

Went through the header files in the pack folder and resolved any unused
header files.

Author: AlexandreSinger

vpr/src/pack/atom_pb_bimap.cpp

@@ -8,7 +8,6 @@
  */
 
 #include "atom_pb_bimap.h"
-#include "atom_netlist.h"
 
 AtomPBBimap::AtomPBBimap(const vtr::bimap<AtomBlockId, const t_pb*, vtr::linear_map, std::unordered_map>& atom_to_pb) {
     atom_to_pb_ = atom_to_pb;

vpr/src/pack/cluster_feasibility_filter.cpp

@@ -28,16 +28,12 @@
  */
 
 #include <vector>
+#include "physical_types.h"
 #include "vtr_assert.h"
 #include "vtr_log.h"
-#include "vtr_memory.h"
 
-#include "read_xml_arch_file.h"
-#include "vpr_types.h"
-#include "globals.h"
 #include "hash.h"
 #include "cluster_feasibility_filter.h"
-#include "vpr_utils.h"
 
 /* header functions that identify pin classes */
 static void alloc_pin_classes_in_pb_graph_node(t_pb_graph_node* pb_graph_node);

vpr/src/pack/cluster_feasibility_filter.h

@@ -21,7 +21,8 @@
 
 #ifndef CLUSTER_FEASIBILITY_CHECK_H
 #define CLUSTER_FEASIBILITY_CHECK_H
-#include "arch_types.h"
+
+class t_pb_graph_node;
 
 void load_pin_classes_in_pb_graph_head(t_pb_graph_node* pb_graph_node);
 

vpr/src/pack/cluster_placement.cpp

@@ -21,7 +21,6 @@
 #include "hash.h"
 #include "physical_types.h"
 #include "prepack.h"
-#include "vpr_types.h"
 #include "vpr_utils.h"
 #include "vtr_assert.h"
 

vpr/src/pack/cluster_router.cpp

@@ -24,7 +24,6 @@
 
 #include "vpr_error.h"
 #include "vpr_types.h"
-#include "echo_files.h"
 
 #include "physical_types.h"
 #include "globals.h"

vpr/src/pack/cluster_router.h

@@ -6,9 +6,13 @@
  */
 #ifndef CLUSTER_ROUTER_H
 #define CLUSTER_ROUTER_H
+
 #include <vector>
 #include "atom_netlist_fwd.h"
+#include "atom_pb_bimap.h"
 #include "pack_types.h"
+#include "vpr_types.h"
+#include "vpr_utils.h"
 
 /* Constructors/Destructors */
 t_lb_router_data* alloc_and_load_router_data(std::vector<t_lb_type_rr_node>* lb_type_graph, t_logical_block_type_ptr type);

vpr/src/pack/cluster_util.cpp

@@ -12,7 +12,6 @@
 #include "prepack.h"
 #include "vpr_context.h"
 #include "vtr_vector.h"
-#include "vtr_vector_map.h"
 
 /*Print the contents of each cluster to an echo file*/
 static void echo_clusters(char* filename, const ClusterLegalizer& cluster_legalizer) {

vpr/src/pack/pack_report.cpp

@@ -2,7 +2,6 @@
 
 #include "vtr_ostream_guard.h"
 
-#include "vpr_types.h"
 #include "vpr_utils.h"
 #include "histogram.h"
 

vpr/src/pack/pack_types.h

@@ -10,9 +10,8 @@
 #include <unordered_map>
 #include <vector>
 
-#include "arch_types.h"
 #include "atom_netlist_fwd.h"
-#include "attraction_groups.h"
+#include "physical_types.h"
 
 struct t_pack_molecule;
 

vpr/src/pack/pb_type_graph.cpp

@@ -13,7 +13,6 @@
 
 #include <cstdio>
 #include <cstring>
-#include <cinttypes>
 #include <queue>
 
 #include "vtr_util.h"
@@ -23,16 +22,13 @@
 #include "vtr_token.h"
 
 #include "vpr_error.h"
-#include "vpr_types.h"
 
 #include "physical_types.h"
 #include "globals.h"
 #include "vpr_utils.h"
 #include "pb_type_graph.h"
 #include "pb_type_graph_annotations.h"
 #include "cluster_feasibility_filter.h"
-#include "power.h"
-#include "read_xml_arch_file.h"
 
 /* variable global to this section that indexes each pb graph pin within a cluster */
 static vtr::t_linked_vptr* edges_head;

vpr/src/pack/pb_type_graph.h

@@ -1,6 +1,8 @@
 #ifndef PB_TYPE_GRAPH_H
 #define PB_TYPE_GRAPH_H
 
+#include "physical_types.h"
+
 struct t_pb_graph_edge_comparator {
     int input_pin_id_in_cluster;
     int output_pin_id_in_cluster;

vpr/src/pack/pb_type_graph_annotations.cpp

@@ -2,24 +2,20 @@
  * April 15, 2011
  * Loads statistical information (min/max delays, power) onto the pb_graph.  */
 
+#include <cmath>
 #include <cstdio>
 #include <cstdlib>
 #include <cstring>
 
+#include "arch_util.h"
 #include "vtr_assert.h"
 #include "vtr_util.h"
-#include "vtr_memory.h"
 #include "vtr_token.h"
 
-#include "vpr_types.h"
 #include "vpr_error.h"
 
-#include "arch_types.h"
-#include "globals.h"
-#include "vpr_utils.h"
 #include "pb_type_graph.h"
 #include "pb_type_graph_annotations.h"
-#include "read_xml_arch_file.h"
 
 static void load_pack_pattern_annotations(const int line_num, t_pb_graph_node* pb_graph_node, const int mode, const char* annot_in_pins, const char* annot_out_pins, const char* value);
 

vpr/src/pack/pb_type_graph_annotations.h

@@ -7,6 +7,8 @@
 #ifndef PB_TYPE_GRAPH_ANNOTATIONS_H
 #define PB_TYPE_GRAPH_ANNOTATIONS_H
 
+class t_pb_graph_node;
+
 void load_pb_graph_pin_to_pin_annotations(t_pb_graph_node* pb_graph_node);
 
 #endif

vpr/src/pack/sync_netlists_to_routing_flat.h

@@ -1,5 +1,3 @@
-#include "netlist.h"
-
 /********************************************************************
  * Top-level function to synchronize packing results to routing results.
  * Flat routing invalidates the ClusteredNetlist since nets may be routed

vpr/src/pack/verify_flat_placement.cpp

@@ -4,7 +4,6 @@
 #include "atom_netlist.h"
 #include "atom_netlist_fwd.h"
 #include "prepack.h"
-#include "vpr_types.h"
 #include "vtr_log.h"
 
 unsigned verify_flat_placement_for_packing(const FlatPlacementInfo& flat_placement_info,