Merge pull request #4140 from The-OpenROAD-Project-staging/secure-fix-dpl-calls

Log for DPL calls

Author: maliberty

flow/scripts/cts.tcl

@@ -46,7 +46,7 @@ set_placement_padding -global \
   -left $::env(CELL_PAD_IN_SITES_DETAIL_PLACEMENT) \
   -right $::env(CELL_PAD_IN_SITES_DETAIL_PLACEMENT)
 
-set result [catch { detailed_placement } msg]
+set result [catch { log_cmd detailed_placement } msg]
 if { $result != 0 } {
   save_progress 4_1_error
   error "Detailed placement failed in CTS: $msg"
@@ -70,7 +70,7 @@ if { !$::env(SKIP_CTS_REPAIR_TIMING) } {
     run_lec_test 4_rsz 4_before_rsz_lec.v 4_after_rsz_lec.v
   }
 
-  set result [catch { detailed_placement } msg]
+  set result [catch { log_cmd detailed_placement } msg]
   if { $result != 0 } {
     save_progress 4_1_error
     error "Detailed placement failed in CTS: $msg"

flow/scripts/detail_place.tcl

@@ -15,7 +15,7 @@ proc do_dpl { } {
   set_placement_padding -global \
     -left $::env(CELL_PAD_IN_SITES_DETAIL_PLACEMENT) \
     -right $::env(CELL_PAD_IN_SITES_DETAIL_PLACEMENT)
-  detailed_placement {*}[env_var_or_empty DETAIL_PLACEMENT_ARGS]
+  log_cmd detailed_placement {*}[env_var_or_empty DETAIL_PLACEMENT_ARGS]
 
   if { $::env(ENABLE_DPO) } {
     if { [env_var_exists_and_non_empty DPO_MAX_DISPLACEMENT] } {

flow/scripts/global_route.tcl

@@ -81,6 +81,7 @@ proc global_route_helper { } {
     # Run to get modified net by DPL
     log_cmd global_route -start_incremental
     log_cmd detailed_placement
+    check_placement -verbose
     # Route only the modified net by DPL
     log_cmd global_route -end_incremental {*}$res_aware \
       -congestion_report_file $::env(REPORTS_DIR)/congestion_post_repair_timing.rpt
@@ -99,6 +100,7 @@ proc global_route_helper { } {
   } {
     puts "Repair antennas..."
     repair_antennas -iterations $::env(MAX_REPAIR_ANTENNAS_ITER_GRT)
+    # repair antennas calls DPL internally
     check_placement -verbose
     check_antennas -report_file $::env(REPORTS_DIR)/grt_antennas.log
   }