flow: Move pre-place buffering from abc to OR floorplan

povik · povik · commit ba891a170611 · 2025-04-23T19:27:10.000+02:00
Signed-off-by: Martin Povišer &lt;povik@cutebit.org&gt;
diff --git a/flow/designs/asap7/minimal/config.mk b/flow/designs/asap7/minimal/config.mk
@@ -19,5 +19,5 @@ export SKIP_REPORT_METRICS = 1
 
 # Faster build, remove these in your own config.mk
 export SKIP_CTS_REPAIR_TIMING = 1
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
 export SKIP_INCREMENTAL_REPAIR = 1
diff --git a/flow/designs/gf12/ariane/config.mk b/flow/designs/gf12/ariane/config.mk
@@ -36,4 +36,4 @@ else
 export DESIGN_TYPE = CELL_NODEN
 endif
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
diff --git a/flow/designs/gf12/ariane133/config.mk b/flow/designs/gf12/ariane133/config.mk
@@ -30,4 +30,4 @@ else
 export DESIGN_TYPE = CELL_NODEN
 endif
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
diff --git a/flow/designs/gf12/swerv_wrapper/config.mk b/flow/designs/gf12/swerv_wrapper/config.mk
@@ -42,4 +42,4 @@ else
 export DESIGN_TYPE = CELL_NODEN
 endif
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
diff --git a/flow/designs/ihp-sg13g2/aes/config.mk b/flow/designs/ihp-sg13g2/aes/config.mk
@@ -13,4 +13,4 @@ export TNS_END_PERCENT = 100
 
 export USE_FILL = 1
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
diff --git a/flow/designs/ihp-sg13g2/jpeg/config.mk b/flow/designs/ihp-sg13g2/jpeg/config.mk
@@ -10,5 +10,5 @@ export CORE_UTILIZATION = 55
 export PLACE_DENSITY_LB_ADDON = 0.20
 export TNS_END_PERCENT = 100
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
 
diff --git a/flow/designs/sky130hd/aes/config.mk b/flow/designs/sky130hd/aes/config.mk
@@ -16,7 +16,7 @@ export TNS_END_PERCENT = 100
 
 export FASTROUTE_TCL = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/fastroute.tcl
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
 
 export CTS_CLUSTER_SIZE = 20
 export CTS_CLUSTER_DIAMETER = 50
diff --git a/flow/designs/sky130hd/ibex/BUILD.bazel b/flow/designs/sky130hd/ibex/BUILD.bazel
@@ -7,7 +7,7 @@ orfs_flow(
         "CORE_UTILIZATION": "45",
         "PLACE_DENSITY_LB_ADDON": "0.2",
         "TNS_END_PERCENT": "100",
-        "REMOVE_ABC_BUFFERS": "1",
+        "FLOORPLAN_BUFFERING": "0",
     },
     pdk = "@docker_orfs//:sky130hd",
     sources = {
diff --git a/flow/designs/sky130hd/ibex/config.mk b/flow/designs/sky130hd/ibex/config.mk
@@ -51,7 +51,7 @@ export TNS_END_PERCENT = 100
 
 export FASTROUTE_TCL = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/fastroute.tcl
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
 
 export CTS_CLUSTER_SIZE = 20
 export CTS_CLUSTER_DIAMETER = 50
diff --git a/flow/designs/sky130hd/jpeg/config.mk b/flow/designs/sky130hd/jpeg/config.mk
@@ -12,4 +12,4 @@ export TNS_END_PERCENT = 100
 
 export FASTROUTE_TCL = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/fastroute.tcl
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
diff --git a/flow/designs/sky130hd/riscv32i/config.mk b/flow/designs/sky130hd/riscv32i/config.mk
@@ -8,4 +8,4 @@ export SDC_FILE      = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/constraint.
 export CORE_UTILIZATION = 45
 export PLACE_DENSITY_LB_ADDON = 0.2
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
diff --git a/flow/designs/sky130hs/aes/config.mk b/flow/designs/sky130hs/aes/config.mk
@@ -12,7 +12,7 @@ export CORE_MARGIN = 2
 export PLACE_DENSITY_LB_ADDON = 0.25
 export TNS_END_PERCENT = 100
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
 
 export CTS_CLUSTER_SIZE = 10
 export CTS_CLUSTER_DIAMETER = 50
diff --git a/flow/designs/sky130hs/ibex/config.mk b/flow/designs/sky130hs/ibex/config.mk
@@ -45,4 +45,4 @@ export CORE_UTILIZATION = 45
 export PLACE_DENSITY_LB_ADDON = 0.2
 export TNS_END_PERCENT = 100
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
diff --git a/flow/designs/sky130hs/riscv32i/config.mk b/flow/designs/sky130hs/riscv32i/config.mk
@@ -13,4 +13,4 @@ export TNS_END_PERCENT = 100
 
 export FASTROUTE_TCL = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/fastroute.tcl
 
-export REMOVE_ABC_BUFFERS = 1
+export FLOORPLAN_BUFFERING = 0
diff --git a/flow/scripts/abc_speed.script b/flow/scripts/abc_speed.script
@@ -28,8 +28,3 @@
 &synch2
 &nf
 &put
-buffer -c
-topo
-stime -c
-upsize -c
-dnsize -c
diff --git a/flow/scripts/floorplan.tcl b/flow/scripts/floorplan.tcl
@@ -40,6 +40,26 @@ check_setup
 set num_instances [llength [get_cells -hier *]]
 puts "number instances in verilog is $num_instances"
 
+# Strip all incoming buffers
+remove_buffers
+
+set_dont_use $::env(DONT_USE_CELLS)
+
+# Do not buffer chip-level designs
+# by default, IO ports will be buffered
+# to not buffer IO ports, set environment variable
+# DONT_BUFFER_PORT = 1
+if { ![env_var_exists_and_non_empty FOOTPRINT] } {
+  if { ![env_var_equals DONT_BUFFER_PORTS 1] } {
+    puts "Perform port buffering..."
+    buffer_ports
+  }
+}
+
+if {[env_var_equals FLOORPLAN_BUFFERING 1]} {
+  repair_design_helper -pre_placement
+}
+
 set additional_args ""
 append_env_var additional_args ADDITIONAL_SITES -additional_sites 1
 
@@ -103,13 +123,6 @@ if {[env_var_exists_and_non_empty FOOTPRINT_TCL]} {
   log_cmd source $::env(FOOTPRINT_TCL)
 }
 
-if { [env_var_equals REMOVE_ABC_BUFFERS 1] } {
-  # remove buffers inserted by yosys/abc
-  remove_buffers
-} else {
-  repair_timing_helper 0
-}
-
 ##### Restructure for timing #########
 if { [env_var_equals RESYNTH_TIMING_RECOVER 1] } {
   repair_design_helper
diff --git a/flow/scripts/resize.tcl b/flow/scripts/resize.tcl
@@ -10,17 +10,6 @@ set pin_count_before [sta::network_leaf_pin_count]
 
 set_dont_use $::env(DONT_USE_CELLS)
 
-# Do not buffer chip-level designs
-# by default, IO ports will be buffered
-# to not buffer IO ports, set environment variable
-# DONT_BUFFER_PORT = 1
-if { ![env_var_exists_and_non_empty FOOTPRINT] } {
-  if { ![env_var_equals DONT_BUFFER_PORTS 1] } {
-    puts "Perform port buffering..."
-    buffer_ports
-  }
-}
-
 repair_design_helper
 
 if { [env_var_exists_and_non_empty TIE_SEPARATION] } {
diff --git a/flow/scripts/util.tcl b/flow/scripts/util.tcl
@@ -40,14 +40,14 @@ proc repair_timing_helper { {hold_margin 1} } {
   log_cmd repair_timing {*}$additional_args
 }
 
-proc repair_design_helper {} {
+proc repair_design_helper { {extra_args ""} } {
   puts "Perform buffer insertion and gate resizing..."
 
-  set additional_args "-verbose"
-  append_env_var additional_args CAP_MARGIN -cap_margin 1
-  append_env_var additional_args SLEW_MARGIN -slew_margin 1
-  append_env_var additional_args MATCH_CELL_FOOTPRINT -match_cell_footprint 0
-  log_cmd repair_design {*}$additional_args
+  set args "-verbose"
+  append_env_var args CAP_MARGIN -cap_margin 1
+  append_env_var args SLEW_MARGIN -slew_margin 1
+  append_env_var args MATCH_CELL_FOOTPRINT -match_cell_footprint 0
+  log_cmd repair_design {*}$args {*}$extra_args
 }
 
 proc recover_power_helper {} {
diff --git a/flow/scripts/variables.yaml b/flow/scripts/variables.yaml
@@ -254,14 +254,12 @@ FLOORPLAN_DEF:
   stages:
     - floorplan
     - place
-REMOVE_ABC_BUFFERS:
+FLOORPLAN_BUFFERING:
   description: >
-    Remove abc buffers from the netlist. If timing repair in floorplanning is
-    taking too long, use a SETUP/HOLD_SLACK_MARGIN to terminate timing repair early
-    instead of using REMOVE_ABC_BUFFERS or set SKIP_LAST_GASP=1.
+    Perform a pre-placement buffering round at the floorplan stage.
   stages:
     - floorplan
-  deprecated: 1
+  default: 1
 PLACE_SITE:
   description: |
     Placement site for core cells defined in the technology LEF file.

Original file line number	Diff line number	Diff line change
`@@ -13,4 +13,4 @@ export TNS_END_PERCENT = 100`
`13`	`13`
`14`	`14`	`export USE_FILL = 1`
`15`	`15`
`16`		`-export REMOVE_ABC_BUFFERS = 1`
	`16`	`+export FLOORPLAN_BUFFERING = 0`
Original file line number	Diff line number	Diff line change
`@@ -12,4 +12,4 @@ export TNS_END_PERCENT = 100`
`12`	`12`
`13`	`13`	`export FASTROUTE_TCL = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/fastroute.tcl`
`14`	`14`
`15`		`-export REMOVE_ABC_BUFFERS = 1`
	`15`	`+export FLOORPLAN_BUFFERING = 0`