Skip to content

placement: run global placement once if IO placement is provided#3379

Closed
oharboe wants to merge 1 commit into
The-OpenROAD-Project:masterfrom
Pinata-Consulting:place-skip-double-global-placement
Closed

placement: run global placement once if IO placement is provided#3379
oharboe wants to merge 1 commit into
The-OpenROAD-Project:masterfrom
Pinata-Consulting:place-skip-double-global-placement

Conversation

@oharboe

@oharboe oharboe commented Aug 4, 2025

Copy link
Copy Markdown
Collaborator

If the computer says yes?

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
@oharboe oharboe force-pushed the place-skip-double-global-placement branch from 4d34c4c to db5de8b Compare August 5, 2025 04:44
@oharboe oharboe requested a review from maliberty August 5, 2025 04:45
@maliberty

Copy link
Copy Markdown
Member

I don't see a reason to merge gpl and ppl into a single step. Just skip that step if needed.

@oharboe

oharboe commented Aug 5, 2025

Copy link
Copy Markdown
Collaborator Author

I don't see a reason to merge gpl and ppl into a single step. Just skip that step if needed.

Is there any reason to run global placement skip io if pins are placed?

To me this is a single concern spread across two stages that should be one stage.

@maliberty

Copy link
Copy Markdown
Member

No but they are not always placed.

@oharboe

oharboe commented Aug 5, 2025

Copy link
Copy Markdown
Collaborator Author

No but they are not always placed.

Lets talk it over in a future call, no rush.

There is some important information I am missing for this to make sense.

@oharboe oharboe closed this Aug 5, 2025
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