Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

cts: add recording of masters used to report_cts #6799

Merged
merged 4 commits into from
Mar 3, 2025
Merged

cts: add recording of masters used to report_cts #6799

merged 4 commits into from
Mar 3, 2025

Conversation

gadfort
Copy link
Collaborator

@gadfort gadfort commented Mar 3, 2025

Changes:

  • no functional changes to CTS, but adds tracking of the masters used during CTS to be able to report that as part of report_cts

gadfort added 2 commits March 3, 2025 09:07
@gadfort
cts: add cells used to cts report
a22e726 
Signed-off-by: Peter Gadfort <peter.gadfort@gmail.com>
@gadfort
cts: add report_cts to some tests
09bd8b4 
Signed-off-by: Peter Gadfort <peter.gadfort@gmail.com>
@gadfort gadfort requested a review from maliberty March 3, 2025 14:13
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Mar 3, 2025

clang-tidy review says "All clean, LGTM! 👍"

maliberty
maliberty reviewed Mar 3, 2025
View reviewed changes
maliberty
maliberty reviewed Mar 3, 2025
View reviewed changes
maliberty
maliberty reviewed Mar 3, 2025
View reviewed changes
@gadfort
cts: break out dummy loads from buffers and use callbacks to track
804890c 
Signed-off-by: Peter Gadfort <peter.gadfort@gmail.com>
@gadfort gadfort requested a review from maliberty March 3, 2025 16:09
maliberty
maliberty requested changes Mar 3, 2025
View reviewed changes
@gadfort
cts: only print dummys if present
b6bb0d1 
Signed-off-by: Peter Gadfort <peter.gadfort@gmail.com>
@gadfort gadfort requested a review from maliberty March 3, 2025 17:20
@maliberty maliberty merged commit 8b0bf81 into The-OpenROAD-Project:master Mar 3, 2025
11 checks passed
@gadfort gadfort deleted the cts-record-masters branch March 3, 2025 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@maliberty maliberty maliberty approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gadfort @maliberty