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

mpl: the distance to boundary is wrongly computed from level != 0 inside SA #6668

Open
AcKoucher opened this issue Feb 7, 2025 · 0 comments · May be fixed by #6689
Open

mpl: the distance to boundary is wrongly computed from level != 0 inside SA #6668

AcKoucher opened this issue Feb 7, 2025 · 0 comments · May be fixed by #6689
Assignees
@AcKoucher
Labels
mpl Macro Placement

Comments

@AcKoucher
Copy link
Contributor

Description

There's an offset compensation missing. This compensation happens "automatically" for all fixed terminals, but, for the unconstrained ios cluster we compute the distance to the boundaries based on the shape of die - which needs to be relocated according to the current parent offset.

Suggested Solution

Apply the correct compensation.

Additional Context

No response
@AcKoucher AcKoucher added the mpl Macro Placement label Feb 7, 2025
@AcKoucher AcKoucher self-assigned this Feb 7, 2025
@AcKoucher AcKoucher linked a pull request Feb 11, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AcKoucher AcKoucher

Labels
mpl Macro Placement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

mpl: fix fixed terminals for cluster of unplaced IOs AcKoucher/OpenROAD
1 participant
@AcKoucher