Skip to content

odb: fix bmap reader to use bterm instead of net since net can have multiple bterms, but bterm can only have one net#9043

Merged
osamahammad21 merged 8 commits intoThe-OpenROAD-Project:masterfrom
gadfort:3dblx-port
Dec 17, 2025
Merged

odb: fix bmap reader to use bterm instead of net since net can have multiple bterms, but bterm can only have one net#9043
osamahammad21 merged 8 commits intoThe-OpenROAD-Project:masterfrom
gadfort:3dblx-port

Conversation

@gadfort
Copy link
Collaborator

@gadfort gadfort commented Dec 15, 2025

Fixes;

  • when synthesis assigns multiple ports to a single net the current code would fail to build the right bpins.

…ultiple bterms, but bterm can only have one net

Signed-off-by: Peter Gadfort <gadfort@zeroasic.com>
@gadfort gadfort requested a review from maliberty December 15, 2025 17:03
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

Signed-off-by: Peter Gadfort <gadfort@zeroasic.com>
@github-actions
Copy link
Contributor

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

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

clang-tidy made some suggestions

Signed-off-by: Peter Gadfort <gadfort@zeroasic.com>
Signed-off-by: Peter Gadfort <gadfort@zeroasic.com>
@github-actions
Copy link
Contributor

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

Signed-off-by: Peter Gadfort <gadfort@zeroasic.com>
@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

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

@osamahammad21 osamahammad21 merged commit 20c0d00 into The-OpenROAD-Project:master Dec 17, 2025
13 checks passed
@gadfort gadfort deleted the 3dblx-port branch December 17, 2025 21:04
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