Skip to content

Conversation

@gadfort
Copy link
Collaborator

@gadfort gadfort commented Nov 13, 2025

Adds:

  • new mode placer to place_pads which implements a general purpose pad placement algorithm.
  • test for placement with placer

Changes:

  • updates single test to add placer on one edge.

Notes:

  • algorithm preserves the order in which pads a requested by the user.
  • avoids obstructions in the rows when possible.

Signed-off-by: Peter Gadfort <gadfort@zeroasic.com>
@gadfort gadfort requested a review from maliberty November 13, 2025 01:32
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! 👍"

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

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

@gadfort gadfort requested a review from maliberty November 13, 2025 13:02
@maliberty maliberty merged commit 8d776b7 into The-OpenROAD-Project:master Nov 13, 2025
13 checks passed
@gadfort gadfort deleted the pad-placer3 branch November 13, 2025 16:29
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