Skip to content

Commit 77e485b

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 0374920 commit 77e485b

2 files changed

Lines changed: 10 additions & 3 deletions

File tree

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)