Merge precisedanon/ORFS-toggle-rebased

Author: PrecisEDAnon

flow/scripts/global_place.tcl

@@ -28,8 +28,20 @@ append_env_var global_placement_args GPL_ROUTABILITY_DRIVEN -routability_driven
 # Parameters for timing driven mode in global placement
 if { $::env(GPL_TIMING_DRIVEN) } {
   lappend global_placement_args {-timing_driven}
-  if { [info exists ::env(GPL_KEEP_OVERFLOW)] } {
-    lappend global_placement_args -keep_resize_below_overflow $::env(GPL_KEEP_OVERFLOW)
+  if { [env_var_truthy ORFS_ENABLE_NEW_OPENROAD] } {
+    set gpl_keep_overflow ""
+    if { [env_var_exists_and_non_empty GPL_KEEP_OVERFLOW] } {
+      set gpl_keep_overflow $::env(GPL_KEEP_OVERFLOW)
+    } elseif { [info exists ::env(PLATFORM)] && $::env(PLATFORM) eq "nangate45" } {
+      set gpl_keep_overflow 1.0
+    }
+    if { $gpl_keep_overflow ne "" } {
+      lappend global_placement_args -keep_resize_below_overflow $gpl_keep_overflow
+    }
+  } else {
+    if { [info exists ::env(GPL_KEEP_OVERFLOW)] } {
+      lappend global_placement_args -keep_resize_below_overflow $::env(GPL_KEEP_OVERFLOW)
+    }
   }
 }
 

flow/scripts/global_route.tcl

@@ -12,9 +12,26 @@ proc global_route_helper { } {
   append_env_var res_aware ENABLE_RESISTANCE_AWARE -resistance_aware 0
 
   proc do_global_route { res_aware } {
+    set default_route_args {-congestion_iterations 30 \
+      -congestion_report_iter_step 5 -verbose}
+    set route_args {}
+
+    if { [env_var_truthy ORFS_ENABLE_NEW_OPENROAD] } {
+      set route_args $default_route_args
+      if { [env_var_exists_and_non_empty GLOBAL_ROUTE_ARGS] } {
+        set route_args [concat $route_args $::env(GLOBAL_ROUTE_ARGS)]
+      }
+    } else {
+      if { [env_var_exists_and_non_empty GLOBAL_ROUTE_ARGS] } {
+        set route_args $::env(GLOBAL_ROUTE_ARGS)
+      } else {
+        set route_args $default_route_args
+      }
+    }
+
     set all_args [concat [list \
       -congestion_report_file $::global_route_congestion_report] \
-      $::env(GLOBAL_ROUTE_ARGS) {*}$res_aware]
+      $route_args {*}$res_aware]
 
     log_cmd global_route {*}$all_args
   }

flow/scripts/load.tcl

@@ -34,6 +34,22 @@ proc load_design { design_file sdc_file } {
   # Read SDC file
   read_sdc $::env(RESULTS_DIR)/$sdc_file
 
+  if { [env_var_truthy ORFS_ENABLE_NEW_OPENROAD]
+       && [info exists ::env(PLATFORM)]
+       && $::env(PLATFORM) eq "nangate45" } {
+    if { ![info exists ::env(NG45_USE_DETAILED_PARA)]
+         || [string trim $::env(NG45_USE_DETAILED_PARA)] eq "" } {
+      # Keep post-route timing consistent with the Nangate45 Oct toolchain by
+      # defaulting to global-route parasitics unless explicitly overridden.
+      set ::env(NG45_USE_DETAILED_PARA) 0
+    }
+    if { [env_var_truthy DISABLE_NG45_DEAD_LOGIC_REMOVAL]
+         && [llength [info commands rsz::set_eliminate_dead_logic_enabled]] } {
+      puts "INFO: Disabling Nangate45 dead-logic elimination."
+      rsz::set_eliminate_dead_logic_enabled 0
+    }
+  }
+
   if { [file exists $::env(PLATFORM_DIR)/derate.tcl] } {
     log_cmd source $::env(PLATFORM_DIR)/derate.tcl
   }

flow/scripts/util.tcl

@@ -15,7 +15,17 @@ proc log_cmd { cmd args } {
 
 proc repair_timing_helper { args } {
   set additional_args {}
-  append_env_var additional_args SETUP_SLACK_MARGIN -setup_margin 1
+  if { [lsearch -exact $args "-setup_margin"] == -1 } {
+    if { [env_var_exists_and_non_empty SETUP_SLACK_MARGIN] } {
+      lappend additional_args -setup_margin $::env(SETUP_SLACK_MARGIN)
+    } else {
+      if { [env_var_truthy ORFS_ENABLE_NEW_OPENROAD] } {
+        lappend additional_args -setup_margin 0.02
+      } else {
+        lappend additional_args -setup_margin 0
+      }
+    }
+  }
   append_env_var additional_args HOLD_SLACK_MARGIN -hold_margin 1
   append_env_var additional_args SETUP_MOVE_SEQUENCE -sequence 1
   append_env_var additional_args TNS_END_PERCENT -repair_tns 1
@@ -107,6 +117,16 @@ proc env_var_exists_and_non_empty { env_var } {
   return [expr { [info exists ::env($env_var)] && ![string equal $::env($env_var) ""] }]
 }
 
+proc env_var_truthy { env_var } {
+  if { [env_var_exists_and_non_empty $env_var] } {
+    set value [string tolower [string trim $::env($env_var)]]
+    if { [lsearch -exact {1 true yes on} $value] != -1 } {
+      return 1
+    }
+  }
+  return 0
+}
+
 proc append_env_var { list_name var_name prefix has_arg } {
   upvar $list_name list
   if {

flow/scripts/variables.mk

@@ -43,6 +43,13 @@ include $(PLATFORM_DIR)/config.mk
 # is no way to escape space in defaults.py and get "foreach" to work.
 $(foreach line,$(shell $(PYTHON_EXE) $(SCRIPTS_DIR)/defaults.py),$(eval export $(subst __SPACE__, ,$(line))))
 
+# Keep hierarchy for large modules when hierarchical synthesis is enabled.
+ifeq ($(strip $(SYNTH_HIERARCHICAL)),1)
+  ifneq (,$(filter 1 true yes on,$(strip $(ORFS_ENABLE_NEW_OPENROAD))))
+    export MAX_UNGROUP_SIZE ?= 50000
+  endif
+endif
+
 #-------------------------------------------------------------------------------
 # DFT / Scan integration (first-class toggle).
 #

flow/scripts/variables.yaml

@@ -22,6 +22,20 @@ GENERATE_ARTIFACTS_ON_FAILURE:
     lot of memory, this allows inspecting results on a laptop for a build that
     ran on a server.
   default: 0
+ORFS_ENABLE_NEW_OPENROAD:
+  description: >
+    When set (1/true/yes/on), enables the experimental OpenROAD/flow behavior
+    on this branch. When unset/0 (default), the flow aims to match vanilla ORFS
+    (93c42b2) + vanilla OpenROAD (7bc521) behavior.
+  stages:
+    - synth
+    - floorplan
+    - place
+    - cts
+    - grt
+    - route
+    - final
+  default: 0
 TNS_END_PERCENT:
   description: >
     Default TNS_END_PERCENT value for post CTS timing repair. Try fixing all
@@ -367,6 +381,14 @@ SYNTH_MINIMUM_KEEP_SIZE:
   stages:
     - synth
   default: 0
+MAX_UNGROUP_SIZE:
+  description: >
+    Area threshold, expressed in NAND2 gate equivalents, used to preserve
+    modules during hierarchical synthesis. When set, this value also seeds
+    SYNTH_MINIMUM_KEEP_SIZE so large blocks are not unintentionally flattened.
+  stages:
+    - synth
+  default: ""
 SYNTH_WRAPPED_OPERATORS:
   description: >
     Synthesize multiple architectural options for each arithmetic operator in the
@@ -656,7 +678,7 @@ SETUP_SLACK_MARGIN:
     - cts
     - floorplan
     - grt
-  default: 0
+  default: ""
 SETUP_REPAIR_SEQUENCE:
   description: |
     Specifies the sequence of moves to do in repair_timing -setup. This should be a string
@@ -1440,10 +1462,21 @@ PRE_GLOBAL_ROUTE_TCL:
     - grt
 GLOBAL_ROUTE_ARGS:
   description: >
-    Replaces default arguments for global route.
+    Extra arguments appended to the default FastRoute invocation.
+    Defaults include -congestion_iterations 30 -congestion_report_iter_step 5 -verbose.
   stages:
     - grt
-  default: -congestion_iterations 30 -congestion_report_iter_step 5 -verbose
+  default: ""
+DISABLE_NG45_DEAD_LOGIC_REMOVAL:
+  description: >
+    Set to 1 to turn off resizer's dead-logic elimination on Nangate45 flows.
+    The default keeps the cleanup enabled while still allowing opt-out for
+    debugging sensitivity to the pass.
+  stages:
+    - floorplan
+    - place
+    - cts
+  default: 0
 MATCH_CELL_FOOTPRINT:
   description: >
     Enforce sizing operations to only swap cells that have the same layout
@@ -1454,6 +1487,17 @@ MATCH_CELL_FOOTPRINT:
     - cts
     - route
   default: 0
+NG45_USE_DETAILED_PARA:
+  description: >
+    Controls which parasitics source resizer should use post-route on
+    Nangate45 designs. The default of 0 keeps the global-route RC model so the
+    timing view aligns with the October toolchain; set to 1 to keep the newer
+    detailed-routing parasitics.
+  stages:
+    - place
+    - cts
+    - grt
+    - route
 RTLMP_MAX_LEVEL:
   description: >
     Maximum depth of the physical hierarchy tree.