[APPack] Fixed issue where density estimation was not used

Author: AlexandreSinger

vpr/src/analytical_place/analytical_placement_flow.cpp

@@ -251,7 +251,8 @@ void run_analytical_placement_flow(t_vpr_setup& vpr_setup) {
     // that downstream stages (e.g. RAM mapper, global placement) can query realistic
     // device dimensions before packing. The packer may later grow or shrink the device
     // size to match the actual resource requirements after packing completes.
-    DeviceSizeEstimator device_size_estimator(vpr_setup, *device_ctx.arch, prepacker);
+    DeviceSizeEstimator device_size_estimator(vpr_setup, *device_ctx.arch, prepacker,
+                                              /*always_estimate_resource_requirement=*/ap_opts.full_legalizer_type == e_ap_full_legalizer::APPack);
 
     // Set up the dedicated clock networks (if used) now that the device grid
     // exists. This must happen before any RR graph is built in this flow

vpr/src/pack/appack_context.cpp

@@ -64,7 +64,6 @@ void APPackContext::adjust_for_device_size_estimate(
 
     if (any_type_needs_denser_packing) {
         VTR_LOG("Device size estimate predicts a tight packing; increased the max candidate distance threshold for the affected block type(s).\n");
-        max_distance_threshold_manager.print_max_dist_thresholds(logical_block_types);
     }
 
     // TODO: This should be capable of turning on unrelated clustering as well if the

vpr/src/pack/appack_max_dist_th_manager.cpp

@@ -39,8 +39,6 @@ void APPackMaxDistThManager::init(const std::vector<std::string>& max_dist_ths,
 
     // Set the initialized flag to true.
     is_initialized_ = true;
-
-    print_max_dist_thresholds(logical_block_types);
 }
 
 void APPackMaxDistThManager::auto_set_max_distance_thresholds(const std::vector<t_logical_block_type>& logical_block_types,

vpr/src/pack/pack.cpp

@@ -358,6 +358,8 @@ bool try_pack(const t_packer_opts& packer_opts,
     e_packer_state current_packer_state = e_packer_state::DEFAULT;
 
     while (current_packer_state != e_packer_state::SUCCESS && current_packer_state != e_packer_state::FAILURE) {
+        if (appack_ctx.appack_options.use_appack)
+            appack_ctx.max_distance_threshold_manager.print_max_dist_thresholds(device_ctx.logical_block_types);
         VTR_LOG("Packing with pin utilization targets: %s\n", cluster_legalizer.get_target_external_pin_util().to_string().c_str());
         VTR_LOG("Packing with high fanout thresholds: %s\n", high_fanout_thresholds.to_string().c_str());
         //Cluster the netlist