Commit Graph

2406 Commits

Author SHA1 Message Date
Jaehyun Kim 2b80a22692 Fix spice tests for upstream write_path_spice API change
Replace removed -spice_directory flag with -spice_file in all 7
spice test scripts to match the upstream API refactoring.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-29 23:02:59 +09:00
Jaehyun Kim 422f774b64 Merge branch 'master' of https://github.com/The-OpenROAD-Project-private/OpenSTA into secure-sta-test-by-opus
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-29 22:33:08 +09:00
Matt Liberty 41ad5c62b8
Merge pull request #322 from The-OpenROAD-Project-staging/sta_update_upstream_lvf_stuff
Sta update upstream lvf stuff
2026-03-27 23:06:30 +00:00
dsengupta0628 ffda41536f consistent signature
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-27 19:53:13 +00:00
dsengupta0628 c5f5468ca0 more update sta fmp and existing spdlog handling
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-27 19:22:27 +00:00
dsengupta0628 676942555c Merge branch 'master' into sta_update_upstream_lvf_stuff 2026-03-27 19:06:05 +00:00
dsengupta0628 4f288f2cf2 Revert unnecessary change in sta/BUILD
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-27 16:28:12 +00:00
Matt Liberty 01a9c69137
Merge pull request #320 from The-OpenROAD-Project-staging/sta_update_upstream_lvf_stuff
fix LibertyParser warnings
2026-03-27 14:56:35 +00:00
dsengupta0628 616280e2f7 fix LibertyParser warnings
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-27 01:10:10 +00:00
Matt Liberty 6771b77a5a
Merge pull request #319 from The-OpenROAD-Project-staging/sta_update_upstream_lvf_stuff
update for succesful bazel build
2026-03-26 22:49:10 +00:00
dsengupta0628 df00dcd224 update for succesful bazel build
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-26 21:11:47 +00:00
Matt Liberty 067cc06e47
Merge pull request #318 from The-OpenROAD-Project-staging/sta_update_upstream_lvf_stuff
Sta update upstream lvf stuff
2026-03-26 15:50:23 +00:00
dsengupta0628 cb8851ef57 Do not clip negative delays just yet- needs more debug
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-26 14:35:25 +00:00
dsengupta0628 d8a92a8c13 Merge branch 'master' into sta_update_upstream_lvf_stuff 2026-03-26 00:38:44 +00:00
dsengupta0628 17febe2c48 update sta code to accomodate LVF and write_spice changes latest on 3/25
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-26 00:38:13 +00:00
Matt Liberty 4b78fcb389
Merge pull request #317 from The-OpenROAD-Project-staging/sta_update_test_infra_to_update_sta_OR
fix warnings
2026-03-25 21:37:25 +00:00
dsengupta0628 35aef386d6 excercise the power func in sta test
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-25 21:09:35 +00:00
dsengupta0628 4238a057dd combined latest osta
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-25 20:03:03 +00:00
dsengupta0628 3f2c80a830 Merge branch 'master' into sta_update_upstream_lvf_stuff 2026-03-25 19:55:19 +00:00
dsengupta0628 9359303763 cleanup more
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-25 18:15:30 +00:00
dsengupta0628 feb392d753 Merge branch 'master' into sta_update_test_infra_to_update_sta_OR 2026-03-25 17:23:05 +00:00
dsengupta0628 2910df0b18 cleanup#2
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-25 16:06:38 +00:00
Matt Liberty ea5531ec19
Merge pull request #316 from The-OpenROAD-Project-staging/sta_update_test_infra_to_update_sta_OR
fix warnings
2026-03-25 15:43:56 +00:00
dsengupta0628 3d5f1e8594 fix warnings
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-03-25 14:48:15 +00:00
James Cherry b9e439f41c delay calc clip delay to 0.0 resolves #405
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-24 18:42:47 -07:00
Robert O'Callahan d233581e16
Make `wire_load_tree_` default to unknown so we can detect when it's not present in a Liberty file. (#402)
This is potentially a behavior change, but I think omitting this is rare. I've only seen it in some DTCD Liberty files.
In those Liberty libraries, it seems to be expected that the Liberty data is valid for all `WireloadTree` types.
Thus it is necessary to distinguish between "wire load tree was specified as 'balanced'" and "wire load tree
was not specified".
2026-03-24 12:35:18 -07:00
James Cherry c38762185c StaConfig.hh pass HAVE_CXX_STD_FORMAT
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-24 12:22:20 -07:00
James Cherry f4048cdf3e read_saif ref missing instance resolves #406
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-24 10:31:24 -07:00
James Cherry 3ec34c571e Only install fmt if std::format is not available
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-24 10:21:44 -07:00
Jaehyun Kim 59663ab640 update test files for upstream API refactoring
Adapt all cpp/tcl test files to match upstream OpenSTA API changes:
- string* output params changed to string return values (Verilog, Network)
- Report methods renamed (reportLineString->reportLine, printf->std::format)
- Delay type refactored from float typedef to class (Graph)
- Liberty model APIs updated (value semantics, shared_ptr, PocvMode)
- EXPECT_STREQ -> EXPECT_EQ for std::string returns (Sdc, Search)
- streamPrint -> sta::print with std::format specifiers (Spice)
- Avoid unset_clock_groups -all path (upstream iterator invalidation bug)

Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-23 13:58:23 +00:00
dsengupta0628 5ce29f4d7d Merge remote-tracking branch 'upstream/master' into sta_update_upstream_lvf_stuff 2026-03-23 13:54:51 +00:00
James Cherry 04f4528c2c ccs ceff pocv
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-22 19:34:59 -07:00
James Cherry ed19ed80df power fmt
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-22 19:34:51 -07:00
James Cherry 7b0281014e regression usage
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-22 10:13:57 -07:00
James Cherry 420519c5d9 write_path_spice
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-21 12:32:10 -07:00
James Cherry bacb61feaa regression -j
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-21 10:18:55 -07:00
Jaehyun Kim 434246a8b4 Fix test suite for upstream API changes after merge
Adapt all C++ and Tcl tests for upstream API refactoring:
- Header rename: DelayFloat.hh -> Delay.hh
- PocvMode enum replacing boolean pocv (setPocvEnabled -> setPocvMode)
- setReportPathFields gains 8th parameter (report_src_attr)
- GateTableModel/CheckTableModel constructor wraps in TableModels
- gateDelay/checkDelay signature changes (bool -> MinMax/PocvMode)
- Unit::asString returns std::string instead of const char*
- ExceptionThru/To::asString removed, use to_string
- GroupPath/FilterPath/LoopPath::asString removed
- PathEnd::source_clk_delay -> source_clk_latency
- report_net -connections flag removed
- set_report_path_field_width removed
- MaxSkewCheck::skew() now requires sta parameter
- Remove tests that pass nullptr to PinIdHash-based maps (segfault)
- Remove tests for removed APIs (sigmaFactor, reportSigmas)
- Regolden .ok files for numerical precision and format changes

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-21 19:23:36 +09:00
Jaehyun Kim 9547beead3 Merge remote-tracking branch 'origin/secure-bump-opensta-0321' into secure-sta-test-by-opus
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-21 18:54:41 +09:00
Jaehyun Kim 43bca8d224 Merge remote-tracking branch 'opensta/master' into secure-sta-test-by-opus
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-21 18:54:37 +09:00
Jaehyun Kim aac881a974 update test files for upstream API refactoring
Adapt all cpp/tcl test files to match upstream OpenSTA API changes:
- string* output params changed to string return values (Verilog, Network)
- Report methods renamed (reportLineString->reportLine, printf->std::format)
- Delay type refactored from float typedef to class (Graph)
- Liberty model APIs updated (value semantics, shared_ptr, PocvMode)
- EXPECT_STREQ -> EXPECT_EQ for std::string returns (Sdc, Search)
- streamPrint -> sta::print with std::format specifiers (Spice)
- Avoid unset_clock_groups -all path (upstream iterator invalidation bug)

Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-21 18:46:31 +09:00
Jaehyun Kim 8868609846 Merge branch 'master' of https://github.com/parallaxsw/OpenSTA
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-21 18:09:38 +09:00
dsengupta0628 dce012ee17 Merge remote-tracking branch 'upstream/master' into sta_update_upstream_lvf_stuff 2026-03-20 19:54:22 +00:00
dsengupta0628 af791b9974 Merge branch 'master' into sta_update_upstream_lvf_stuff 2026-03-20 19:50:24 +00:00
James Cherry d90bf7d93b rename Report virtuals to warnMsg etc
This reverts commit 2ed1c2c06b.

Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-20 10:14:57 -07:00
James Cherry 2ed1c2c06b report json
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-20 09:11:39 -07:00
James Cherry 07c7eeb624 Report::warn etc virtuals
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-20 09:05:07 -07:00
Jaehyun Kim 9b7c99de85 test: fix StaTclInterpAccess failure in Release builds
Sta::tcl_interp_ is not initialized in the constructor, so
tclInterp() returns garbage before setTclInterp() is called.
The assertion `before == nullptr` fails in Release builds where
memory is not zero-initialized.

Remove the read of uninitialized tcl_interp_ and the assertion.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-20 23:37:35 +09:00
Jaehyun Kim 57e264f252 test: remove tests that pass nullptr Pin to PinIdHash-based maps
PinIdHash::operator()(nullptr) calls network_->id(nullptr) which
dereferences null, causing segfault in CI environments.

Removed 3 tests:
- SdcInitTest.SdcIsLeafPinNonGeneratedClockNull
- StaInitTest.IsClockSrcNoDesign
- StaInitTest.StaIsClockSrcNoDesign2

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-20 23:11:25 +09:00
Jaehyun Kim 0364faddd7 fix regression failures after master merge
- Remove deprecated report_net -connections/-verbose flags from tests
  (removed upstream in 7668e43d)
- Regolden numerical precision diffs in parasitics and util tests

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-20 22:15:51 +09:00
Jaehyun Kim 992a1dcec5 Merge branch 'master' of https://github.com/The-OpenROAD-Project-private/OpenSTA into secure-sta-test-by-opus
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-03-20 21:56:12 +09:00