luke
/
OpenSTA
mirror of https://github.com/The-OpenROAD-Project/OpenSTA.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity
2,429 Commits 1 Branch 1 Tag 249 MiB
Commit Graph

2429 Commits

Author SHA1 Message Date
Jaehyun Kim d3454bc464 Stabilize tests against non-deterministic ordering and version changes 
- Use diff_files_sorted for exception_override_priority SDC comparisons
  to handle non-deterministic write_sdc output ordering across environments
- Add (VERSION ignore pattern to SDF diff_files calls so tests don't
  break on upstream version bumps

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
 2026-04-02 10:07:23 +09:00
Jaehyun Kim 61bdb83038 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-04-01 23:54:49 +09:00
Matt Liberty 12683d645c
Merge pull request #325 from The-OpenROAD-Project-staging/sta_latest_merge_strings 
update swig file to not use deprecated api and not test it either
 2026-03-31 19:02:49 +00:00
dsengupta0628 b37b37e0d9 update swig file to not use deprecated api and not test it either 
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
 2026-03-31 15:50:43 +00:00
Jaehyun Kim 14af24bc93 Ignore SDF VERSION in diff_files to stabilize tests 
Add (VERSION pattern to diff_files ignore parameter so SDF tests
don't break on every upstream version bump. Update golden .ok files
to reflect the now-clean diffs.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
 2026-03-31 22:05:59 +09:00
Jaehyun Kim 3ca78b11c8 Adapt test files to upstream API changes (string_view migration) 
Update 16 test files to match upstream OpenSTA API refactoring:
- const char* → std::string/std::string_view across all test APIs
- EXPECT_STREQ → EXPECT_EQ for std::string/string_view returns
- nullptr → "" for string_view parameters (makeClock, makeClockGroups, etc.)
- PropertyValue("literal") → PropertyValue(std::string("literal")) to avoid
  bool constructor overload resolution
- Method renames: relatedGroundPin→relatedGroundPort,
  relatedPowerPin→relatedPowerPort, firstName→firstParam,
  secondName→secondParam
- Constructor changes: OperatingConditions (5-arg → 1-arg + setters),
  ModeDef::defineValue (3-arg → 1-arg + setSdfCond)
- LibertyAttrValue::floatValue returns std::pair<float,bool> now
- Filter API: separate args → single expression string + bool_props_as_int
- SDF version golden files: 3.0.1 → 3.1.0

All 6104 tests pass.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
 2026-03-31 16:34:40 +09:00
Jaehyun Kim b42e064281 Merge remote-tracking branch 'opensta/master' into secure-sta-test-by-opus 
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>

# Conflicts:
#	test/get_filter.ok
 2026-03-31 15:29:44 +09:00
Jaehyun Kim 069de0498c 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-31 15:27:01 +09:00
Matt Liberty fe61456df6
Merge pull request #324 from The-OpenROAD-Project-staging/sta_latest_merge_strings 
Sta latest merge strings
 2026-03-31 02:00:13 +00:00
dsengupta0628 6ca7cb55a4 update test infra 2 
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
 2026-03-30 21:49:10 +00:00
dsengupta0628 b5f647cb7f update test infra to accomodate std::string instead of const char* 
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
 2026-03-30 19:44:26 +00:00
James Cherry 5955958564 findScaleFactorType, findScaleFactorPvt string_view 
Signed-off-by: James Cherry <cherry@parallaxsw.com>
 2026-03-30 11:01:15 -07:00
James Cherry 538db6211f filter_objects via Akash Levy resolves #399 2026-03-30 09:36:21 -07:00
dsengupta0628 5f46ad3dfb Merge remote-tracking branch 'upstream/master' into sta_latest_merge_strings 2026-03-30 16:22:27 +00:00
James Cherry 46472e0eed regresssion update failures/diffs 
Signed-off-by: James Cherry <cherry@parallaxsw.com>
 2026-03-29 15:19:00 -07:00
James Cherry 5aa56bcfee cmake swig 
Signed-off-by: James Cherry <cherry@parallaxsw.com>
 2026-03-29 11:23:26 -07:00
James Cherry 725a6e5e1c require cmake 3.16 
Signed-off-by: James Cherry <cherry@parallaxsw.com>
 2026-03-29 10:56:45 -07:00
James Cherry 3d0b34e88c ReportPath::reportLine 
Signed-off-by: James Cherry <cherry@parallaxsw.com>
 2026-03-29 09:58:06 -07:00
Jaehyun Kim 0a3368e177 Fix non-deterministic SDC write ordering in tests 
Use diff_files_sorted for SDC golden file comparisons in
disable_case and write_roundtrip_full tests. The upstream
set_min_pulse_width output ordering is non-deterministic
due to hash map iteration, causing intermittent CI failures.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
 2026-03-29 23:31:33 +09:00
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
James Cherry 1806e4aede StaTclTypes %include "std_string_view.i" 
Signed-off-by: James Cherry <cherry@parallaxsw.com>
 2026-03-28 19:55:13 -07:00
James Cherry 614385fe51 Liberty unnecessary move 
Signed-off-by: James Cherry <cherry@parallaxsw.com>
 2026-03-28 19:48:21 -07:00
James Cherry ad80dd55b3 cmake show swig version 
Signed-off-by: James Cherry <cherry@parallaxsw.com>
 2026-03-28 19:46:57 -07:00
James Cherry 6742692876 string squash 
Signed-off-by: James Cherry <cherry@parallaxsw.com>
 2026-03-28 19:13:35 -07: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