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.

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

detailed_placement creates unroutable placement for centered floorplan #6704

Open
stefanottili opened this issue Feb 12, 2025 · 3 comments
Open

Comments

@stefanottili
Copy link

stefanottili commented Feb 12, 2025

Describe the bug

revisiting #5557 shows that gpl/rudy work correctly for a centered floorplan.

It's the detailed_placement that messes up the stdcells and creates an unroutable placement.

Expected Behavior

detailed_placement should be able to handle any DIEAREA ( x1 y1 ) ( x2 y2 ), not just x1 = 0, y1 = 0, x2 > 0 y2 > 0

Environment

MacOS M1 homebrew OpenROAD v2.0-18955-g3332c5ee6

To Reproduce

#5557 has a tar file with lef/def/or command file.

Relevant log output

[NesterovSolve] Finished with Overflow: 0.099042
Placement Analysis
---------------------------------
total displacement    3613512.4 u
average displacement      110.8 u
max displacement          736.0 u
original HPWL         1091240.1 u
legalized HPWL        4943157.6 u
delta HPWL                  353 %

Screenshots

No response

Additional Context

No response

@stefanottili
Copy link
Author

Same netlist using a first quadrant floorplan works as expected.

Placement Analysis
---------------------------------
total displacement     107598.5 u
average displacement        3.3 u
max displacement           15.1 u
original HPWL         1091253.2 u
legalized HPWL        1194890.2 u
delta HPWL                    9 %

@stefanottili
Copy link
Author

stefanottili commented Feb 13, 2025

detailed_placement worked as of filing #5557 in Aug 15 2024.
bisecting is tricky since older versions don’t compile with boost 1.87

@maliberty
Copy link
Member

Don't bother bisecting as there were substantial changes for hybrid rows. This has no priority to fix

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

No branches or pull requests

2 participants