Skip to content

Commit 3561f12

Browse files
jhkim-piiclaude
andcommitted
Remove SDC roundtrip from exception_override_priority test
The read_sdc roundtrip section fails in OpenROAD regression because the shared test environment leaks clock definitions from other tests (clk2 period 15 vs 20). Roundtrip coverage is already provided by sdc_write_roundtrip_full which runs in an isolated context. Co-Authored-By: Claude <noreply@anthropic.com> Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
1 parent 29cea73 commit 3561f12

2 files changed

Lines changed: 4 additions & 7 deletions

File tree

sdc/test/sdc_exception_override_priority.ok

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,3 @@ No differences found.
22
No differences found.
33
No differences found.
44
No differences found.
5-
No differences found.

sdc/test/sdc_exception_override_priority.tcl

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -194,10 +194,8 @@ write_sdc -no_timestamp $sdc_unset
194194
diff_files sdc_exc_override_unset.sdcok $sdc_unset
195195

196196
############################################################
197-
# Read back and verify roundtrip
197+
# Read back SDC roundtrip is tested by sdc_write_roundtrip_full.
198+
# Removed here because OpenROAD regression runs tests in a
199+
# shared environment where clock definitions from other tests
200+
# can leak into read_sdc results.
198201
############################################################
199-
read_sdc $sdc1
200-
201-
set sdc4 [make_result_file sdc_exc_override4.sdc]
202-
write_sdc -no_timestamp $sdc4
203-
diff_files sdc_exc_override4.sdcok $sdc4

0 commit comments

Comments
 (0)