Commit Graph

2517 Commits

Author SHA1 Message Date
James Cherry 2f85fdfee3 Dmp cleanup
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-03 16:46:05 -07:00
James Cherry 6bb888b218 stringFloat comment
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-03 11:25:33 -07:00
James Cherry e04f36b541 GraphDelayCalc::findInputArcDelay rm Slew
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-03 11:22:38 -07:00
James Cherry 5d40134239 ArnoldiDelayCalc rcmodel
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-03 11:22:38 -07:00
James Cherry f873c6520a Library/Cell name/filename args string_view
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-04-03 11:22:38 -07:00
Mohamed Gaber a5921d1ca9
util/StringUtil.cc: feature-test std::from_chars (#415)
Support for std::{from,to}_chars isn't finalized in
libcxx as of the time of writing, see __cpp_lib_to_chars in
6331bfa41a/libcxx/docs/FeatureTestMacroTable.rst

This patch adds a fallback using strtof. There are two
differences:
* strtof is locale-dependent
* strtof tolerates leading spaces

Signed-off-by: Mohamed Gaber <me@donn.website>
2026-04-03 11:21:57 -07:00
Henner Zeller e8218f2db1
Tcl 9 compatibility changes (#414)
* Provide close2proc function to prevent tcl9 from crashing.

Tcl 9 does not test if the close2Proc function pointer is non-null,
but calls it unconditionally:
https://github.com/tcltk/tcl/blob/core-9-0-3/generic/tclIO.c#L384

So we need to provide a non-null function pointer for our code
to not crash with Tcl9.

Use the same implementation as the previous close channel
had.

Signed-off-by: Henner Zeller <h.zeller@acm.org>

* Use non-deprecated trace add variable syntax.

In modern tcl, `trace variable` is now i`trace add variable`,
and `"rw"` should be spelled out as `{read write}`
There were backwards compatible forms in Tcl 8.x but now loudly
complains in Tcl 9

Signed-off-by: Henner Zeller <h.zeller@acm.org>

* Use `Tcl_Size` for all tcl functions returning sizes.

This is the type the Tcl-API provides in its prototypes and
starting from Tcl9 this typedef actually changes from `int` to `long`,
so will no longer compile when passing an `int*`.

So whenever we get a return value of this type, use the
correct typedef to declare the variable. This makes it forward and
backward compatible.

Signed-off-by: Henner Zeller <h.zeller@acm.org>

* Address review comments: compare with `read`/`write` not `r`, `w`

Signed-off-by: Henner Zeller <h.zeller@acm.org>

---------

Signed-off-by: Henner Zeller <h.zeller@acm.org>
2026-04-03 11:02:25 -07:00
Matt Liberty 8b4c581a90
Merge pull request #330 from The-OpenROAD-Project-staging/fix-exception-override-priority
Fix exception_override_priority test for OpenROAD regression
2026-04-03 04:45:06 +00:00
Jaehyun Kim 7d0e0bb641 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-03 12:27:03 +09:00
Jaehyun Kim 619d6bc6fa Merge branch 'fix-exception-override-priority' into secure-test-suite-cleanup
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-03 12:21:10 +09:00
Jaehyun Kim 1d38466776 Remove dead roundtrip comment blocks from SDC tests
Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-03 12:17:55 +09:00
Jaehyun Kim 05e65b1dbf Remove read_sdc roundtrip from all SDC tests
OpenROAD regression runs 7600+ tests in a shared environment where
clock definitions leak between tests. Any test using read_sdc picks
up contaminated state, causing spurious failures (e.g., clk2 period
15 vs 20). Roundtrip coverage is provided by sdc_write_roundtrip_full
which runs in isolation.

Affected tests (19 total):
  exception_override_priority, exception_thru_override,
  exception_intersect, exception_thru_complex,
  exception_merge_priority, exception_rise_fall_transitions,
  exception_match_filter, exception_advanced,
  delay_borrow_group, design_rules_limits, drive_input_pvt,
  net_wire_voltage, capacitance_propagated, removal_reset,
  write_disabled_groups, clock_operations, clock_removal_cascade,
  write_options, write_comprehensive, sense_unset_override

All 6107 tests pass.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-03 11:02:08 +09:00
Mike Inouye 4180288868
Add string_view header and sort includes in ContainerHelpers.hh (#413) 2026-04-02 15:59:01 -07:00
Matt Liberty 4c8efd7474
Merge pull request #306 from The-OpenROAD-Project/dependabot/github_actions/actions/cache-5
Bump actions/cache from 4 to 5
2026-04-02 21:23:59 +00:00
Matt Liberty f0dd97833a
Merge pull request #305 from The-OpenROAD-Project/dependabot/github_actions/actions/checkout-6
Bump actions/checkout from 5 to 6
2026-04-02 21:23:19 +00:00
Matt Liberty 94bb37bb4a
Merge pull request #327 from The-OpenROAD-Project/driver-vertex-level-cache
added driver vertex levelization
2026-04-02 21:22:03 +00:00
Jaehyun Kim a1c1685c04 Clean up test suite per 3-way code review findings
- Remove temp files: network_gcd_traversal.log, util_report_redirect.log
- Delete 19 comment-only C++ test stubs (dead code for removed APIs)
- Remove redundant graph_make_verify test (covered by graph_advanced)
- Centralize assert_file_nonempty/assert_file_contains into test/helpers.tcl
  and remove inline copies from 17 verilog test files
- Fix Build.sh stray quotes in heredoc output
- Fix regression.sh unquoted variable expansions
- Update .gitignore: add */test/*.log and Testing/

All 6087 tests pass.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-02 22:27:27 +09:00
Jaehyun Kim 3561f12c7a 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>
2026-04-02 21:44:26 +09:00
Matt Liberty 29cea73c3d
Merge pull request #294 from The-OpenROAD-Project-staging/secure-sta-test-by-opus
Add new Tcl and C++ test cases
2026-04-02 12:18:17 +00:00
Jaehyun Kim bb186c39dd Remove temporary files
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-02 20:45:42 +09:00
Cho Moon 748fb9bd4b code review changes per Deepashree
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
2026-04-02 06:27:17 +00:00
Jaehyun Kim f192996a42 Update golden files for upstream bias_pins feature
Upstream added bias pin type recognition, changing VNB/VPB pin
direction from "unknown" to "bias" in liberty output. Update 9
golden .ok files to match.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-02 12:42:08 +09:00
Jaehyun Kim a22112fe8f Merge remote-tracking branch 'origin/master' into secure-sta-test-by-opus
# Conflicts:
#	verilog/test/CMakeLists.txt

Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-02 12:30:18 +09:00
Jaehyun Kim ea22794652 Merge remote-tracking branch 'opensta/master' into secure-sta-test-by-opus
# Conflicts:
#	liberty/TableModel.cc

Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-02 12:21:14 +09:00
Matt Liberty 438633dfa8
Merge pull request #329 from The-OpenROAD-Project-staging/sta_bias_pin_issue_commit_history_correct
Sta bias pin issue handling, non-determinism in delay calc
2026-04-02 03:06:57 +00:00
Jaehyun Kim 0e5f14eb37 Revert "Stabilize tests against non-deterministic ordering and version changes"
This reverts commit d3454bc464.

Signed-off-by: Jaehyun Kim <jhkim@precisioninno.com>
2026-04-02 11:06:36 +09:00
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
dsengupta0628 d47cbeb76b fix to exclude bias pins from timing graph
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-02 00:56:38 +00:00
dsengupta0628 5e2d5e8ad5 pull latest master, has delay calc non-determinism fix
Signed-off-by: dsengupta0628 <dsengupta@precisioninno.com>
2026-04-02 00:49:47 +00: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
James Cherry 70659c2328 PathVisitor::visitFaninPaths filter check edges
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-31 21:03:55 -07:00
Cho Moon 211aea7497 added one more Sta API
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
2026-04-01 00:12:26 +00:00
James Cherry 638ffa57d4 cleanup
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-31 16:45:47 -07:00
James Cherry 8c1ebce695 GraphDelayCalc::findInputDriverDelay rm finishDrvrPin
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-31 16:37:38 -07:00
Cho Moon 55bb77ff89 added driver vertex levelization
Signed-off-by: Cho Moon <cmoon@precisioninno.com>
2026-03-31 22:59:52 +00:00
James Cherry 9eb9edb0b3 GateTableModel::gateDelay do not clip delays
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-31 14:20:50 -07:00
James Cherry 1d4b516239 GraphDelayCalc::findInputArcDelay thread issue
Signed-off-by: James Cherry <cherry@parallaxsw.com>
2026-03-31 13:59:59 -07: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