Skip to content

grt: use minimum pin layer to create guides for local nets#9975

Merged
eder-matheus merged 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:grt_local_nets
Mar 27, 2026
Merged

grt: use minimum pin layer to create guides for local nets#9975
eder-matheus merged 1 commit into
The-OpenROAD-Project:masterfrom
The-OpenROAD-Project-staging:grt_local_nets

Conversation

@openroad-ci
Copy link
Copy Markdown
Collaborator

@openroad-ci openroad-ci commented Mar 27, 2026

Summary

When creating guides for local nets, the minimum layer was always set to the first metal layer. This PR changes it to use the minimum pin layer between the net pins.

Type of Change

  • Bug fix

Impact

Correct creation of guide stack for local nets.

Verification

  • I have verified that the local build succeeds (./etc/Build.sh).
  • I have run the relevant tests and they pass.
  • My code follows the repository's formatting guidelines.
  • I have signed my commits (DCO).

Related Issues

Address Discussion #9814

Signed-off-by: Eder Monteiro <emrmonteiro@precisioninno.com>
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

Gemini is experiencing higher than usual traffic and was unable to create the review. Please try again in a few hours by commenting /gemini review.

@github-actions
Copy link
Copy Markdown
Contributor

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

@eder-matheus eder-matheus requested a review from maliberty March 27, 2026 12:14
@eder-matheus eder-matheus enabled auto-merge March 27, 2026 15:05
@eder-matheus
Copy link
Copy Markdown
Member

@maliberty This is ready to merge, and shouldn't affect our CI.

@eder-matheus eder-matheus merged commit e9f1e18 into The-OpenROAD-Project:master Mar 27, 2026
15 checks passed
@openroad-ci openroad-ci deleted the grt_local_nets branch March 27, 2026 23:16
donn added a commit to librelane/librelane that referenced this pull request Apr 2, 2026
See The-OpenROAD-Project/OpenROAD#9814.

Also added a number of missing Qt plugins since we're rebuilding OpenROAD anyway so might as well.
donn added a commit to librelane/librelane that referenced this pull request Apr 2, 2026
See The-OpenROAD-Project/OpenROAD#9814.

Also added a number of missing Qt plugins since we're rebuilding
OpenROAD anyway so might as well.
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.

3 participants