Skip to content

Commit 8724066

Browse files
oharboeclaude
andcommitted
fix: separate EQUIVALENCE_CHECK from RUN_EQY to avoid requiring eqy
EQUIVALENCE_CHECK now only controls writing equivalence check files. The new RUN_EQY variable (default 0) gates actually executing the eqy tool, so builds don't fail when eqy is not installed. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
1 parent f5b9c3f commit 8724066

3 files changed

Lines changed: 13 additions & 4 deletions

File tree

docs/user/FlowVariables.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ configuration file.
137137
| <a name="DPO_MAX_DISPLACEMENT"></a>DPO_MAX_DISPLACEMENT| Specifies how far an instance can be moved when optimizing.| 5 1|
138138
| <a name="EARLY_SIZING_CAP_RATIO"></a>EARLY_SIZING_CAP_RATIO| Ratio between the input pin capacitance and the output pin load during initial gate sizing.| |
139139
| <a name="ENABLE_DPO"></a>ENABLE_DPO| Enable detail placement with improve_placement feature.| 1|
140-
| <a name="EQUIVALENCE_CHECK"></a>EQUIVALENCE_CHECK| Enable running equivalence checks to verify logical correctness of repair_timing.| 0|
140+
| <a name="EQUIVALENCE_CHECK"></a>EQUIVALENCE_CHECK| Enable writing equivalence check files to verify logical correctness of repair_timing. Set RUN_EQY to actually run the eqy tool.| 0|
141141
| <a name="FASTROUTE_TCL"></a>FASTROUTE_TCL| Specifies a Tcl script with commands to run before FastRoute.| |
142142
| <a name="FILL_CELLS"></a>FILL_CELLS| Fill cells are used to fill empty sites. If not set or empty, fill cell insertion is skipped.| |
143143
| <a name="FILL_CONFIG"></a>FILL_CONFIG| JSON rule file for metal fill during chip finishing.| |
@@ -253,6 +253,7 @@ configuration file.
253253
| <a name="RTLMP_RPT_DIR"></a>RTLMP_RPT_DIR| Path to the directory where reports are saved.| |
254254
| <a name="RTLMP_WIRELENGTH_WT"></a>RTLMP_WIRELENGTH_WT| Weight for half-perimiter wirelength.| 100.0|
255255
| <a name="RULES_JSON"></a>RULES_JSON| json files with the metrics baseline regression rules. In the ORFS Makefile, this defaults to $DESIGN_DIR/rules-base.json, but ORFS does not mandate the users source directory layout and this can be placed elsewhere when the user sets up an ORFS config.mk or from bazel-orfs.| |
256+
| <a name="RUN_EQY"></a>RUN_EQY| Actually run the eqy equivalence checking tool. Requires EQUIVALENCE_CHECK to be enabled and eqy to be installed.| 0|
256257
| <a name="RUN_LOG_NAME_STEM"></a>RUN_LOG_NAME_STEM| Stem of the log file name, the log file will be named `$(LOG_DIR)/$(RUN_LOG_NAME_STEM).log`.| run|
257258
| <a name="RUN_SCRIPT"></a>RUN_SCRIPT| Path to script to run from `make run`, python or tcl script detected by .py or .tcl extension.| |
258259
| <a name="SC_LEF"></a>SC_LEF| Path to technology standard cell LEF file.| |
@@ -489,6 +490,7 @@ configuration file.
489490
- [PRE_CTS_TCL](#PRE_CTS_TCL)
490491
- [REMOVE_CELLS_FOR_EQY](#REMOVE_CELLS_FOR_EQY)
491492
- [REPORT_CLOCK_SKEW](#REPORT_CLOCK_SKEW)
493+
- [RUN_EQY](#RUN_EQY)
492494
- [SETUP_REPAIR_SEQUENCE](#SETUP_REPAIR_SEQUENCE)
493495
- [SETUP_SLACK_MARGIN](#SETUP_SLACK_MARGIN)
494496
- [SKIP_CRIT_VT_SWAP](#SKIP_CRIT_VT_SWAP)

flow/scripts/cts.tcl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ if { !$::env(SKIP_CTS_REPAIR_TIMING) } {
6868

6969
repair_timing_helper
7070

71-
if { $::env(EQUIVALENCE_CHECK) } {
71+
if { $::env(EQUIVALENCE_CHECK) && $::env(RUN_EQY) } {
7272
run_equivalence_test
7373
}
7474
if { $::env(LEC_CHECK) } {

flow/scripts/variables.yaml

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -86,8 +86,15 @@ DETAILED_METRICS:
8686
- grt
8787
EQUIVALENCE_CHECK:
8888
description: >
89-
Enable running equivalence checks to verify logical correctness of
90-
repair_timing.
89+
Enable writing equivalence check files to verify logical correctness of
90+
repair_timing. Set RUN_EQY to actually run the eqy tool.
91+
default: 0
92+
stages:
93+
- cts
94+
RUN_EQY:
95+
description: >
96+
Actually run the eqy equivalence checking tool. Requires EQUIVALENCE_CHECK
97+
to be enabled and eqy to be installed.
9198
default: 0
9299
stages:
93100
- cts

0 commit comments

Comments
 (0)