-
Notifications
You must be signed in to change notification settings - Fork 415
Dense initial placement #2113
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
Dense initial placement #2113
Conversation
…een fixed with new dense placement approach
Looks 3 symbiflow circuits are failing with "No routing path exists" for a special connection. Maybe some special IO placement code, or respecting locked down blocks, got changed accidentally? |
Good to do a QoR run to be safe (vtr and titan) and attach results. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the helper functions! Some comments to address.
Here is the comparison between new version and baseline results for three benchmarks. |
Looks good. The QoR looks fine; there are some ups and downs in cpu time but that appears to be due to machine load as sometimes the new code is slower and sometimes it is faster, and there are changes in various parts of the flow (pack, place, route, even abc). Will merge once CI passes. |
Initial placement used to fail in case of high utilization or long chains. New approach called dense_placement has been added to initial placement, make it possible to redo initial placement if random placement could not find a legal solution for high utilization designs.
Relate issue : [https://github.com//issues/2036#issuecomment-1184739027](Initial Placement issue)
List of Design that passing now:
@MohamedElgammal @vaughnbetz