Skip to content

sky130hd/microwatt remove antenna iterations limit - #4748

Open
joaomai wants to merge 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:sky130hd-microwatt-antenna-iter
Open

joaomai wants to merge 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:sky130hd-microwatt-antenna-iter

Conversation

@joaomai

@joaomai joaomai commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

The limit was added due to the large runtime introduced by a bug in pin-aware macro channels. Since the bug is now fixed, it can be restored. Detailed route runtime increases in local runs.

Metric Before After
detailedroute__antenna__violating__nets 96 4
detailedroute__antenna__violating__pins 114 5
detailedroute__antenna_diodes_count 1800 2175
runtime (s) 2054 3252

Signed-off-by: João Mai <jmai@precisioninno.com>
@joaomai joaomai self-assigned this Oct 27, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request removes the MAX_REPAIR_ANTENNAS_ITER_DRT environment variable export from the config.mk configuration file for the sky130hd/microwatt design. There are no review comments, and I have no feedback to provide.

@joaomai
joaomai requested a review from jfgava October 27, 2026 16:15
@joaomai
joaomai requested review from eder-matheus and removed request for eder-matheus October 28, 2026 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants