[fix] update graphic structure call moved

Now a full legalize method is created to update drawing cluster
data structure. It is called at the end of each legalize implementation.

Author: robluo

vpr/src/analytical_place/full_legalizer.cpp

@@ -59,6 +59,9 @@
 
 #include "setup_grid.h"
 #include "stats.h"
+#ifndef NO_GRAPHICS
+#include "draw_global.h"
+#endif
 
 std::unique_ptr<FullLegalizer> make_full_legalizer(e_ap_full_legalizer full_legalizer_type,
                                                    const APNetlist& ap_netlist,
@@ -1003,6 +1006,7 @@ void FlatRecon::legalize(const PartialPlacement& p_placement) {
 
     // Perform the initial placement on created clusters.
     place_clusters(p_placement);
+    update_drawing_data_structures();
 }
 
 void NaiveFullLegalizer::create_clusters(const PartialPlacement& p_placement) {
@@ -1219,6 +1223,7 @@ void NaiveFullLegalizer::legalize(const PartialPlacement& p_placement) {
     //       made part of the placement and verify placement should check for
     //       it.
     post_place_sync();
+    update_drawing_data_structures();
 }
 
 void APPack::legalize(const PartialPlacement& p_placement) {
@@ -1304,4 +1309,14 @@ void APPack::legalize(const PartialPlacement& p_placement) {
 
     // Synchronize the pins in the clusters after placement.
     post_place_sync();
+    update_drawing_data_structures();
 }
+
+void FullLegalizer::update_drawing_data_structures() {
+#ifndef NO_GRAPHICS
+    // update graphic resources incase of clustering changes
+    if (get_draw_state_vars()) {
+        get_draw_state_vars()->refresh_graphic_resources_after_cluster_change();
+    }
+#endif
+}

vpr/src/analytical_place/full_legalizer.h

@@ -59,6 +59,9 @@ class FullLegalizer {
      *                      device grid and fixed blocks are observed.
      */
     virtual void legalize(const PartialPlacement& p_placement) = 0;
+    
+    /// @brief Update drawing data structure for current placement
+    void update_drawing_data_structures();
 
   protected:
     /// @brief The AP Netlist to fully legalize the flat placement of.

vpr/src/base/place_and_route.cpp

@@ -20,9 +20,6 @@
 #include "route.h"
 #include "route_export.h"
 #include "draw.h"
-#ifndef NO_GRAPHICS
-#include "draw_global.h"
-#endif
 #include "rr_graph.h"
 #include "read_xml_arch_file.h"
 #include "route_common.h"
@@ -607,11 +604,4 @@ void post_place_sync() {
     for (const ClusterBlockId block_id : cluster_ctx.clb_nlist.blocks()) {
         blk_loc_registry.place_sync_external_block_connections(block_id);
     }
-
-#ifndef NO_GRAPHICS
-    // update graphic resources incase of clustering changes
-    if (get_draw_state_vars()) {
-        get_draw_state_vars()->refresh_graphic_resources_after_cluster_change();
-    }
-#endif
 }