Fixed CTS_CLUSTER_SIZE & CTS_CLUSTER_DIAMETER handling

updated sky130hd microwatt CTS cluster variables
updated rules

Signed-off-by: Jeff Ng <[email protected]>

Author: jeffng-or

flow/designs/gf180/ibex/rules-base.json

@@ -48,23 +48,23 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -1.58,
+        "value": -1.53,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 899459,
+        "value": 1082396,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
-        "value": 1013,
+        "value": 799,
         "compare": "<="
     },
     "finish__timing__drv__hold_violation_count": {
         "value": 100,
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
-        "value": -21.3,
+        "value": -20.21,
         "compare": ">="
     }
 }

flow/designs/gf180/jpeg/rules-base.json

@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 15,
+        "value": 10,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -48,11 +48,11 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.08,
+        "value": -0.51,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 2438978,
+        "value": 2837982,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {

flow/designs/gf180/riscv32i/rules-base.json

@@ -52,7 +52,7 @@
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 435127,
+        "value": 521614,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {

flow/designs/nangate45/swerv/rules-base.json

@@ -48,7 +48,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.37,
+        "value": -0.35,
         "compare": ">="
     },
     "finish__design__instance__area": {
@@ -60,7 +60,7 @@
         "compare": "<="
     },
     "finish__timing__drv__hold_violation_count": {
-        "value": 281,
+        "value": 584,
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {

flow/designs/sky130hd/microwatt/config.mk

@@ -38,6 +38,8 @@ export PLACE_DENSITY = 0.21
 # CTS tuning
 export CTS_BUF_DISTANCE = 600
 export SKIP_GATE_CLONING = 1
+export CTS_CLUSTER_SIZE = 10
+export CTS_CLUSTER_DIAMETER = 50
 
 export SETUP_SLACK_MARGIN = 0.2
 

flow/designs/sky130hd/microwatt/rules-base.json

@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 3810,
+        "value": 3594,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -60,7 +60,7 @@
         "compare": "<="
     },
     "finish__timing__drv__hold_violation_count": {
-        "value": 106,
+        "value": 269,
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {

flow/designs/sky130hd/riscv32i/rules-base.json

@@ -44,15 +44,15 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 21,
+        "value": 5,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
         "value": -1.36,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 90213,
+        "value": 104628,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {

flow/designs/sky130hs/ibex/rules-base.json

@@ -28,11 +28,11 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 30,
+        "value": 22,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
-        "value": 787855,
+        "value": 917273,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
@@ -44,11 +44,11 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 38,
+        "value": 34,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.46,
+        "value": -0.43,
         "compare": ">="
     },
     "finish__design__instance__area": {

flow/designs/sky130hs/riscv32i/rules-base.json

@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 3,
+        "value": 2,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -52,7 +52,7 @@
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 124092,
+        "value": 143440,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {

flow/scripts/cts.tcl

@@ -19,10 +19,8 @@ set cts_args [list \
           -balance_levels]
 
 append_env_var cts_args CTS_BUF_DISTANCE -distance_between_buffers 1
-# TODO: The next two are no-ops since the arg order is wrong, but hard to get
-# through CI since nine designs change metrics and the PR is blocked
-append_env_var cts_args -sink_clustering_size CTS_CLUSTER_SIZE 1
-append_env_var cts_args -sink_clustering_max_diameter CTS_CLUSTER_DIAMETER 1
+append_env_var cts_args CTS_CLUSTER_SIZE -sink_clustering_size 1
+append_env_var cts_args CTS_CLUSTER_DIAMETER -sink_clustering_max_diameter 1
 append_env_var cts_args CTS_BUF_LIST -buf_list 1
 
 if {[env_var_exists_and_non_empty CTS_ARGS]} {