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

update sky130hd/microwatt place density #2558

Merged
merged 6 commits into from
Nov 14, 2024
Merged

update sky130hd/microwatt place density #2558

merged 6 commits into from
Nov 14, 2024
+311 −316

Conversation

eder-matheus
Copy link
Collaborator

@eder-matheus eder-matheus commented Nov 12, 2024 •
edited
Loading

Metric Old New Type
placeopt__design__instance__count__stdcell 191879 191836 Tighten
cts__design__instance__count__setup_buffer 16685 16681 Tighten
cts__design__instance__count__hold_buffer 16685 16681 Tighten
globalroute__antenna_diodes_count 7292 8305 Failing
detailedroute__route__wirelength 9705104 10332428 Failing
detailedroute__antenna__violating__nets 3 0 Tighten
detailedroute__antenna_diodes_count 1420 1213 Tighten
finish__timing__drv__setup_violation_count 8343 8341 Tighten

maliberty
maliberty approved these changes Nov 12, 2024
View reviewed changes
Copy link
Member

@maliberty maliberty left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does this affect the runtime?

@maliberty
Copy link
Member

This fails in uW:

13:29:50  [INFO GRT-0018] Total wirelength: 11589619 um
13:29:50  [INFO GRT-0014] Routed nets: 89816
13:29:50  [ERROR GRT-0116] Global routing finished with congestion. Check the congestion regions in the DRC Viewer.
13:29:50  Error: global_route.tcl, 110 GRT-0116

@eder-matheus
Copy link
Collaborator Author

@maliberty The runtime of microwatt was largely decreased, taking less than 3 hours to complete when running in my local machine. I don't get why it fails with congestion in the ci, but I'll investigate better.

@eder-matheus eder-matheus merged commit b28c142 into The-OpenROAD-Project:master Nov 14, 2024
6 checks passed
@eder-matheus eder-matheus deleted the update_microwatt branch November 18, 2024 15:11
@eder-matheus eder-matheus restored the update_microwatt branch November 18, 2024 15:12
@eder-matheus eder-matheus deleted the update_microwatt branch November 18, 2024 15:12
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
@eder-matheus @maliberty