Merge remote-tracking branch 'pinata/synth-even-better-names' into synth-names-and-canonicalize

Signed-off-by: Matt Liberty <[email protected]>

Author: maliberty

flow/designs/asap7/gcd/metadata-base-ok.json

@@ -8,19 +8,19 @@
   "cts__cpu__total": 6.83,
   "cts__design__core__area": 197.122,
   "cts__design__die__area": 262.44,
-  "cts__design__instance__area": 53.9023,
+  "cts__design__instance__area": 53.3191,
   "cts__design__instance__area__macros": 0,
-  "cts__design__instance__area__stdcell": 53.9023,
-  "cts__design__instance__count": 541,
+  "cts__design__instance__area__stdcell": 53.3191,
+  "cts__design__instance__count": 549,
   "cts__design__instance__count__hold_buffer": 0,
   "cts__design__instance__count__macros": 0,
-  "cts__design__instance__count__setup_buffer": 48,
-  "cts__design__instance__count__stdcell": 541,
-  "cts__design__instance__displacement__max": 1.023,
-  "cts__design__instance__displacement__mean": 0.033,
-  "cts__design__instance__displacement__total": 18.136,
-  "cts__design__instance__utilization": 0.273447,
-  "cts__design__instance__utilization__stdcell": 0.273447,
+  "cts__design__instance__count__setup_buffer": 58,
+  "cts__design__instance__count__stdcell": 549,
+  "cts__design__instance__displacement__max": 1.205,
+  "cts__design__instance__displacement__mean": 0.048,
+  "cts__design__instance__displacement__total": 26.725,
+  "cts__design__instance__utilization": 0.270488,
+  "cts__design__instance__utilization__stdcell": 0.270488,
   "cts__design__io": 54,
   "cts__design__violations": 0,
   "cts__flow__errors__count": 0,
@@ -34,7 +34,7 @@
   "cts__runtime__total": "0:06.93",
   "cts__timing__drv__hold_violation_count": 0,
   "cts__timing__drv__max_cap": 0,
-  "cts__timing__drv__max_cap_limit": 0.805526,
+  "cts__timing__drv__max_cap_limit": 0.738979,
   "cts__timing__drv__max_fanout": 0,
   "cts__timing__drv__max_fanout_limit": 0,
   "cts__timing__drv__max_slew": 0,
@@ -46,17 +46,17 @@
   "detailedplace__cpu__total": 2.28,
   "detailedplace__design__core__area": 197.122,
   "detailedplace__design__die__area": 262.44,
-  "detailedplace__design__instance__area": 49.6012,
+  "detailedplace__design__instance__area": 48.2161,
   "detailedplace__design__instance__area__macros": 0,
-  "detailedplace__design__instance__area__stdcell": 49.6012,
-  "detailedplace__design__instance__count": 485,
+  "detailedplace__design__instance__area__stdcell": 48.2161,
+  "detailedplace__design__instance__count": 483,
   "detailedplace__design__instance__count__macros": 0,
-  "detailedplace__design__instance__count__stdcell": 485,
-  "detailedplace__design__instance__displacement__max": 0.827,
-  "detailedplace__design__instance__displacement__mean": 0.148,
-  "detailedplace__design__instance__displacement__total": 71.879,
-  "detailedplace__design__instance__utilization": 0.251627,
-  "detailedplace__design__instance__utilization__stdcell": 0.251627,
+  "detailedplace__design__instance__count__stdcell": 483,
+  "detailedplace__design__instance__displacement__max": 1.109,
+  "detailedplace__design__instance__displacement__mean": 0.168,
+  "detailedplace__design__instance__displacement__total": 81.354,
+  "detailedplace__design__instance__utilization": 0.244601,
+  "detailedplace__design__instance__utilization__stdcell": 0.244601,
   "detailedplace__design__io": 54,
   "detailedplace__design__violations": 0,
   "detailedplace__flow__errors__count": 0,
@@ -70,7 +70,7 @@
   "detailedplace__runtime__total": "0:02.38",
   "detailedplace__timing__drv__hold_violation_count": 0,
   "detailedplace__timing__drv__max_cap": 0,
-  "detailedplace__timing__drv__max_cap_limit": 0.701391,
+  "detailedplace__timing__drv__max_cap_limit": 0.694704,
   "detailedplace__timing__drv__max_fanout": 0,
   "detailedplace__timing__drv__max_fanout_limit": 0,
   "detailedplace__timing__drv__max_slew": 0,
@@ -85,13 +85,12 @@
   "detailedroute__flow__warnings__count": 15,
   "detailedroute__mem__peak": 1641060.0,
   "detailedroute__route__drc_errors": 0,
-  "detailedroute__route__drc_errors__iter:1": 149,
-  "detailedroute__route__drc_errors__iter:2": 8,
-  "detailedroute__route__drc_errors__iter:3": 1,
-  "detailedroute__route__drc_errors__iter:4": 0,
-  "detailedroute__route__net": 503,
+  "detailedroute__route__drc_errors__iter:1": 58,
+  "detailedroute__route__drc_errors__iter:2": 7,
+  "detailedroute__route__drc_errors__iter:3": 0,
+  "detailedroute__route__net": 516,
   "detailedroute__route__net__special": 2,
-  "detailedroute__route__vias": 3518,
+  "detailedroute__route__vias": 3629,
   "detailedroute__route__vias__multicut": 0,
   "detailedroute__route__vias__singlecut": 3518,
   "detailedroute__route__wirelength": 1145,
@@ -108,14 +107,14 @@
   "finish__cpu__total": 4.21,
   "finish__design__core__area": 197.122,
   "finish__design__die__area": 262.44,
-  "finish__design__instance__area": 54.0626,
+  "finish__design__instance__area": 53.8002,
   "finish__design__instance__area__macros": 0,
-  "finish__design__instance__area__stdcell": 54.0626,
-  "finish__design__instance__count": 542,
+  "finish__design__instance__area__stdcell": 53.8002,
+  "finish__design__instance__count": 555,
   "finish__design__instance__count__macros": 0,
-  "finish__design__instance__count__stdcell": 542,
-  "finish__design__instance__utilization": 0.27426,
-  "finish__design__instance__utilization__stdcell": 0.27426,
+  "finish__design__instance__count__stdcell": 555,
+  "finish__design__instance__utilization": 0.272929,
+  "finish__design__instance__utilization__stdcell": 0.272929,
   "finish__design__io": 54,
   "finish__design_powergrid__drop__average__net:VDD__corner:default": 0.768654,
   "finish__design_powergrid__drop__average__net:VSS__corner:default": 0.00159977,
@@ -133,7 +132,7 @@
   "finish__runtime__total": "0:04.45",
   "finish__timing__drv__hold_violation_count": 0,
   "finish__timing__drv__max_cap": 0,
-  "finish__timing__drv__max_cap_limit": 0.806972,
+  "finish__timing__drv__max_cap_limit": 0.75093,
   "finish__timing__drv__max_fanout": 0,
   "finish__timing__drv__max_fanout_limit": 0,
   "finish__timing__drv__max_slew": 0,
@@ -148,14 +147,14 @@
   "floorplan__cpu__total": 2.16,
   "floorplan__design__core__area": 197.122,
   "floorplan__design__die__area": 262.44,
-  "floorplan__design__instance__area": 41.0427,
+  "floorplan__design__instance__area": 40.3866,
   "floorplan__design__instance__area__macros": 0,
-  "floorplan__design__instance__area__stdcell": 41.0427,
-  "floorplan__design__instance__count": 328,
+  "floorplan__design__instance__area__stdcell": 40.3866,
+  "floorplan__design__instance__count": 326,
   "floorplan__design__instance__count__macros": 0,
-  "floorplan__design__instance__count__stdcell": 328,
-  "floorplan__design__instance__utilization": 0.20821,
-  "floorplan__design__instance__utilization__stdcell": 0.20821,
+  "floorplan__design__instance__count__stdcell": 326,
+  "floorplan__design__instance__utilization": 0.204882,
+  "floorplan__design__instance__utilization__stdcell": 0.204882,
   "floorplan__design__io": 54,
   "floorplan__flow__errors__count": 0,
   "floorplan__flow__warnings__count": 10,
@@ -187,14 +186,14 @@
   "globalplace__cpu__total": 3.17,
   "globalplace__design__core__area": 197.122,
   "globalplace__design__die__area": 262.44,
-  "globalplace__design__instance__area": 44.0753,
+  "globalplace__design__instance__area": 43.4192,
   "globalplace__design__instance__area__macros": 0,
-  "globalplace__design__instance__area__stdcell": 44.0753,
-  "globalplace__design__instance__count": 432,
+  "globalplace__design__instance__area__stdcell": 43.4192,
+  "globalplace__design__instance__count": 430,
   "globalplace__design__instance__count__macros": 0,
-  "globalplace__design__instance__count__stdcell": 432,
-  "globalplace__design__instance__utilization": 0.223595,
-  "globalplace__design__instance__utilization__stdcell": 0.223595,
+  "globalplace__design__instance__count__stdcell": 430,
+  "globalplace__design__instance__utilization": 0.220266,
+  "globalplace__design__instance__utilization__stdcell": 0.220266,
   "globalplace__design__io": 54,
   "globalplace__flow__errors__count": 0,
   "globalplace__flow__warnings__count": 10,
@@ -219,19 +218,19 @@
   "globalroute__cpu__total": 5.41,
   "globalroute__design__core__area": 197.122,
   "globalroute__design__die__area": 262.44,
-  "globalroute__design__instance__area": 54.0626,
+  "globalroute__design__instance__area": 53.8002,
   "globalroute__design__instance__area__macros": 0,
-  "globalroute__design__instance__area__stdcell": 54.0626,
-  "globalroute__design__instance__count": 542,
+  "globalroute__design__instance__area__stdcell": 53.8002,
+  "globalroute__design__instance__count": 555,
   "globalroute__design__instance__count__hold_buffer": 0,
   "globalroute__design__instance__count__macros": 0,
-  "globalroute__design__instance__count__setup_buffer": 0,
-  "globalroute__design__instance__count__stdcell": 542,
-  "globalroute__design__instance__displacement__max": 0,
-  "globalroute__design__instance__displacement__mean": 0,
-  "globalroute__design__instance__displacement__total": 0,
-  "globalroute__design__instance__utilization": 0.27426,
-  "globalroute__design__instance__utilization__stdcell": 0.27426,
+  "globalroute__design__instance__count__setup_buffer": 6,
+  "globalroute__design__instance__count__stdcell": 555,
+  "globalroute__design__instance__displacement__max": 0.594,
+  "globalroute__design__instance__displacement__mean": 0.005,
+  "globalroute__design__instance__displacement__total": 3.294,
+  "globalroute__design__instance__utilization": 0.272929,
+  "globalroute__design__instance__utilization__stdcell": 0.272929,
   "globalroute__design__io": 54,
   "globalroute__design__violations": 0,
   "globalroute__flow__errors__count": 0,
@@ -246,7 +245,7 @@
   "globalroute__timing__clock__slack": -11.536,
   "globalroute__timing__drv__hold_violation_count": 0,
   "globalroute__timing__drv__max_cap": 0,
-  "globalroute__timing__drv__max_cap_limit": 0.799574,
+  "globalroute__timing__drv__max_cap_limit": 0.746669,
   "globalroute__timing__drv__max_fanout": 0,
   "globalroute__timing__drv__max_fanout_limit": 0,
   "globalroute__timing__drv__max_slew": 0,
@@ -257,14 +256,14 @@
   "placeopt__cpu__total": 2.41,
   "placeopt__design__core__area": 197.122,
   "placeopt__design__die__area": 262.44,
-  "placeopt__design__instance__area": 49.6012,
+  "placeopt__design__instance__area": 48.2161,
   "placeopt__design__instance__area__macros": 0,
-  "placeopt__design__instance__area__stdcell": 49.6012,
-  "placeopt__design__instance__count": 485,
+  "placeopt__design__instance__area__stdcell": 48.2161,
+  "placeopt__design__instance__count": 483,
   "placeopt__design__instance__count__macros": 0,
-  "placeopt__design__instance__count__stdcell": 485,
-  "placeopt__design__instance__utilization": 0.251627,
-  "placeopt__design__instance__utilization__stdcell": 0.251627,
+  "placeopt__design__instance__count__stdcell": 483,
+  "placeopt__design__instance__utilization": 0.244601,
+  "placeopt__design__instance__utilization__stdcell": 0.244601,
   "placeopt__design__io": 54,
   "placeopt__flow__errors__count": 0,
   "placeopt__flow__warnings__count": 10,
@@ -278,14 +277,14 @@
   "placeopt__timing__drv__floating__pins": 0,
   "placeopt__timing__drv__hold_violation_count": 0,
   "placeopt__timing__drv__max_cap": 0,
-  "placeopt__timing__drv__max_cap_limit": 0.702876,
+  "placeopt__timing__drv__max_cap_limit": 0.696302,
   "placeopt__timing__drv__max_fanout": 0,
   "placeopt__timing__drv__max_fanout_limit": 0,
   "placeopt__timing__drv__max_slew": 0,
-  "placeopt__timing__drv__max_slew_limit": 0.573761,
-  "placeopt__timing__drv__setup_violation_count": 35,
-  "placeopt__timing__setup__tns": -369.188,
-  "placeopt__timing__setup__ws": -23.8066,
+  "placeopt__timing__drv__max_slew_limit": 0.56476,
+  "placeopt__timing__drv__setup_violation_count": 21,
+  "placeopt__timing__setup__tns": -263.203,
+  "placeopt__timing__setup__ws": -28.9187,
   "run__flow__design": "gcd",
   "run__flow__generate_date": "2024-06-20 21:30",
   "run__flow__metrics_version": "Metrics_2.1.2",
@@ -309,4 +308,4 @@
   "synth__mem__peak": 154440.0,
   "synth__runtime__total": "0:06.19",
   "total_time": "0:01:20.650000"
-}
+}

flow/designs/asap7/gcd/rules-base.json

@@ -20,15 +20,15 @@
         "compare": "=="
     },
     "cts__design__instance__count__setup_buffer": {
-        "value": 53,
+        "value": 64,
         "compare": "<="
     },
     "cts__design__instance__count__hold_buffer": {
         "value": 44,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
-        "value": 1317,
+        "value": 1316,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {

flow/scripts/synth.tcl

@@ -14,6 +14,11 @@ if { [info exist ::env(SYNTH_GUT)] && $::env(SYNTH_GUT) == 1 } {
 
 synthesize_check $::env(SYNTH_FULL_ARGS)
 
+# rename registers to have the verilog register name in its name
+# of the form \regName$_DFF_P_. We should fix yosys to make it the reg name.
+# At least this is predictable.
+renames -wire
+
 if { [info exists ::env(USE_LSORACLE)] } {
     set lso_script [open $::env(OBJECTS_DIR)/lso.script w]
     puts $lso_script "ps -a"
@@ -47,11 +52,6 @@ if {[info exist ::env(LATCH_MAP_FILE)]} {
   techmap -map $::env(LATCH_MAP_FILE)
 }
 
-# rename registers to have the verilog register name in its name
-# of the form \regName$_DFF_P_. We should fix yosys to make it the reg name.
-# At least this is predictable.
-renames -wire
-
 set dfflibmap_args ""
 foreach cell $::env(DONT_USE_CELLS) {
   lappend dfflibmap_args -dont_use $cell