Commit Graph

2517 Commits

Author SHA1 Message Date
Waylander c3455612d8
Merge 335b305149 into a56edf2767 2026-04-12 04:05:42 -03:00
Matt Liberty a56edf2767
Merge pull request #345 from The-OpenROAD-Project-staging/sta_statetable_mbff_regression
Restore statetable check in hasSequentials and add MBFF regression test
2026-04-11 21:43:59 +00:00
dsengupta0628 097daeb3c5 address comment related to Coverity to simplify
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-10 18:59:42 +00:00
dsengupta0628 5878d2594d Dont update the swig as we are not adding any TCL regression so not needed
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-10 15:53:55 +00:00
dsengupta0628 44643b5733 fix state table mbff issue and coverity in this file
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-10 15:44:30 +00:00
Matt Liberty defec0bda3
Merge pull request #344 from The-OpenROAD-Project-staging/sta_mbff_changes
Show original name of pins in path report for mbff cluster…
2026-04-09 22:12:27 +00:00
dsengupta0628 ca4e757828 Merge branch 'master' into sta_mbff_changes 2026-04-08 22:01:20 +00:00
Matt Liberty 6599d47d27
Merge pull request #343 from The-OpenROAD-Project-staging/sta_latest_from_parallaxsw_0407
Sta latest from parallaxsw 0407
2026-04-08 20:05:51 +00:00
dsengupta0628 37520c08d2 changes to show original name of pins in path report for mbff clustered flops
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-08 15:39:18 +00:00
dsengupta0628 d7d732932a remove unnecessary files and update .gititnore
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-08 00:28:53 +00:00
dsengupta0628 3b8e81393d Merge branch 'master' into sta_latest_from_parallaxsw_0407 2026-04-07 21:53:36 +00:00
dsengupta0628 98835d8f20 update tests
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-07 21:47:01 +00:00
dsengupta0628 371b85cd20 latest STA plus changes to fix prima dcalc and read_spef issues, TCL regression added
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-07 20:48:15 +00:00
Deepashree Sengupta c887b2e4b3
Bias pin handling (#409)
* Update STA to exclude bias pins from timing graph and subsequently in write_verilog

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

* unnecessary space in orig verilog

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

* Update to use well supplies rather than bias pins

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>

---------

Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-07 11:00:01 -07:00
James Cherry 645f2669c9 report_dcalc prima resolves #418
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-07 10:45:04 -07:00
Matt Liberty 12bbfed551
Merge pull request #335 from hzeller/feature-20260405-init-tcl
Use same tcl initialization as in OpenROAD
2026-04-07 16:13:08 +00:00
Matt Liberty 46622306ba
Merge branch 'master' into feature-20260405-init-tcl 2026-04-07 15:56:49 +00:00
Matt Liberty f54e2aed94
Merge pull request #340 from The-OpenROAD-Project-staging/secure-revert-helper-change
Revert diff_files/diff_files_sorted to original return behavior
2026-04-07 15:21:38 +00:00
Jaehyun Kim 5fd751e8d7 Revert diff_files/diff_files_sorted to original return behavior
Revert error-on-mismatch back to return 1, as the new GitHub Action
(github-actions-are-differences-found) now detects "Differences found"
in CI output, making the error approach unnecessary.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-07 23:42:21 +09:00
Matt Liberty d19c99932f
Merge pull request #332 from The-OpenROAD-Project-staging/secure-test-suite-cleanup
Clean up test suite: remove dead code, fix race conditions, centralize helpers
2026-04-07 14:10:28 +00:00
dsengupta0628 3eec04f4e8 update STA to Parallaxsw version on 4/7 with fix for issue 416 and some unrelated stuff
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-07 12:59:25 +00:00
Jaehyun Kim ba46d45157 Merge branch 'master' of https://github.com/The-OpenROAD-Project-private/OpenSTA into secure-test-suite-cleanup
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-07 09:52:59 +09:00
Matt Liberty e4338cf834
Merge pull request #339 from The-OpenROAD-Project-staging/sta_handle_funny_defs
fix: guard parseBusName against non-numeric bus subscripts
2026-04-06 17:57:24 +00:00
dsengupta0628 2a5592122d Update unguarded stoi handling
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-06 16:38:58 +00:00
Matt Liberty 7b97ee46a6
Merge branch 'master' into feature-20260405-init-tcl 2026-04-06 15:22:22 +00:00
Jaehyun Kim cf1db116dc Remove dead re-add constraint blocks from SDC tests
removeConstraints was removed upstream in STA 3.0 (MCMM refactor).
The "Re-add constraints" blocks were originally preceded by
remove_constraints calls; without them, the re-adds are no-ops.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-07 00:06:42 +09:00
Jaehyun Kim bc7639be11 Revert CMakeLists.txt -Werror changes
Drop the ALLOW_WARNINGS option, -Werror flag, and
OpenSTAWarnings interface library. Restore original
target_compile_options with generator expressions.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-06 23:41:51 +09:00
Jaehyun Kim 616cb6234a Fix report_checks -fields {nets} typo to {net} across test scripts
The valid field name is "net" (singular) per search/Search.tcl.
Using "nets" triggered Warning 168 (unknown field) silently in
29 test scripts. Fix the field name and regolden .ok files.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-06 14:41:40 +09:00
Jaehyun Kim b30c82b2d0 Make diff_files/diff_files_sorted error on mismatch
Previously these functions returned 1 on mismatch but no caller
checked the return value, allowing roundtrip failures to go
undetected. Now they call error() so the sta process exits
non-zero and the regression runner catches the failure.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-06 14:35:59 +09:00
Jaehyun Kim b6429ffe95 Remove dead remove_constraints placeholder blocks from SDC tests
The remove_constraints API was permanently removed from Sta.
Delete the commented-out calls, TODO comments, and "skipped"
print statements instead of keeping placeholder blocks.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-06 14:35:53 +09:00
Jaehyun Kim d6a80441e4 Merge branch 'master' of https://github.com/The-OpenROAD-Project-private/OpenSTA into secure-test-suite-cleanup
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-06 14:35:47 +09:00
Matt Liberty f8bf0fd0e2
Merge pull request #336 from The-OpenROAD-Project-staging/add-difference-found
Add GHA to check for differences found in .ok files
2026-04-06 05:19:01 +00:00
Matt Liberty d1bc00cf8c update tests to avoid "Differences found" in .ok files
Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
2026-04-06 04:58:47 +00:00
Matt Liberty 687314a055 Add GHA to check for differences found in .ok files
Same as in OR

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
2026-04-06 04:58:47 +00:00
Matt Liberty 5c0158bad9
Merge pull request #337 from The-OpenROAD-Project-staging/secure-fix-test-diffs
Regolden test files to fix false diff_files mismatches
2026-04-06 04:57:51 +00:00
Jaehyun Kim 52887aa6db Regolden test golden files to fix false diff_files mismatches
Update .libok/.txtok/.sdcok golden files to match current writer
output so diff_files produces "No differences found" instead of
silently reporting mismatches that were captured as expected output.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-06 10:19:05 +09:00
Jaehyun Kim 0c15e18f94 build: Add -Werror by default
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-05 23:55:35 +09:00
Jaehyun Kim 1170781f9a Merge remote-tracking branch 'origin/master' into secure-test-suite-cleanup
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-05 22:42:35 +09:00
Henner Zeller 385f4fdb79 Use same tcl initialization as in OpenROAD
In https://github.com/The-OpenROAD-Project/OpenROAD/pull/10041
we have a new way to initializa the tcl subsystem which
will allow to be entirely independent of separate files
once we can switch to Tcl9.

Signed-off-by: Henner Zeller <h.zeller@acm.org>
2026-04-05 08:53:54 +02:00
Matt Liberty 3d2bcb1dd1
Merge pull request #334 from The-OpenROAD-Project-staging/sta_changes_to_fix_test
Sta changes to fix test
2026-04-05 01:55:05 +00:00
dsengupta0628 43177bba8f more to fix
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-05 00:38:29 +00:00
dsengupta0628 fc7ac6c16c test/cpp changes
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-05 00:18:45 +00:00
James Cherry 548b665412 get_* -filter allow true/false, '.' in glob pattern resolves #416
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-04 16:19:41 -07:00
James Cherry d6e826ef8b save_ok hook
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-04 14:58:25 -07:00
James Cherry e3d1bb09b0 Network [[nodiscard]]
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-04 09:26:03 -07:00
Matt Liberty d6c13c424a
Merge pull request #333 from The-OpenROAD-Project-staging/filter-dot
In FilterExpr::lex allow '.' in key = value filter
2026-04-04 16:21:55 +00:00
James Cherry 84150e925b prima ceff
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-04 09:21:34 -07:00
James Cherry aedddcecd8 DMP use std::array
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-04 09:21:01 -07:00
Matt Liberty e288db0225 In FilterExpr::lex allow '.' in key = value filter
The current lexer doesn't allow the . in the filter
  name =~ tcdm_master_.*req_.*_i

Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
2026-04-04 15:23:04 +00:00
James Cherry fe23c4530f regression.tcl
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-03 16:46:13 -07:00