-
Notifications
You must be signed in to change notification settings - Fork 339
flow: Move pre-place buffering from abc to OR floorplan #3101
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
base: master
Are you sure you want to change the base?
flow: Move pre-place buffering from abc to OR floorplan #3101
Conversation
Signed-off-by: Martin Povišer <povik@cutebit.org>
ba891a1
to
9d56d8e
Compare
Secure run at secure-floorplan-buffering |
@@ -40,14 +40,14 @@ proc repair_timing_helper { {hold_margin 1} } { | |||
log_cmd repair_timing {*}$additional_args | |||
} | |||
|
|||
proc repair_design_helper {} { | |||
proc repair_design_helper { {extra_args ""} } { |
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.
Just a heads up, used in global_route.tcl too. Should be fine near as I can see.
$ grep -r repair_design_helper scripts/*
scripts/floorplan.tcl: repair_design_helper
scripts/floorplan.tcl: repair_design_helper
scripts/global_route.tcl: repair_design_helper
scripts/resize.tcl:repair_design_helper
scripts/util.tcl:proc repair_design_helper {} {
Non-metric failures from public CI:
this has the symptoms of The-OpenROAD-Project/OpenROAD#6991
|
Curious failure mode from macro placer on gf12/bp_single. @AcKoucher is there anything I should do to triage? |
@povik I think this might be related with the approach we're using to deal with PADs in MPL - regarding how we map PADs with the internal objects of MPL. There is a PR that changes the current approach. Could you try using those changes? I just updated it. |
No description provided.