[rsz] Change RSZ-0075 warning to debug + remove pre-placement buffer attempt #3900
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -130,7 +130,6 @@ if { !$::env(SKIP_REPAIR_TIE_FANOUT) } { | |
| } | ||
|
|
||
| if { [env_var_exists_and_non_empty SWAP_ARITH_OPERATORS] } { | ||
| replace_arith_modules | ||
| } | ||
|
|
||
|
|
@@ -139,7 +138,7 @@ if { $::env(REMOVE_ABC_BUFFERS) } { | |
| remove_buffers | ||
| } else { | ||
| # Skip clone & split | ||
| repair_timing_helper -setup -skip_last_gasp -sequence "unbuffer,sizeup,swap,vt_swap" | ||
|
Contributor
There was a problem hiding this comment. @povik I removed
Contributor
There was a problem hiding this comment. We remove buffers at 3_2 so any buffering we do at floorplan will be short-lived. Perhaps it's useful to do it anyway to get a more accurate timing report and to guide other resizer operations which look at timing. Matt and I have discussed this a couple of times but never did any changes as it always seemed low priority.
Contributor
There was a problem hiding this comment. Thank you for sharing. @precisionmoon @maliberty Any other thoughts?
Member
There was a problem hiding this comment. I think it currently does little as most moves abort when there is no placement. If a secure CI confirms I am ok to remove it. The original thought was to use gain-based buffering here.
Contributor
There was a problem hiding this comment. OK. Then let's merge this PR after resolving the merge conflict caused by MCMM update. I'll check if the gain-based buffering in floorplan stage is meaningful gain or not, and open a new PR if it shows meaningful gain. |
||
| } | ||
|
|
||
| puts "Default units for flow" | ||
|
|
||
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.
Unplaced at this stage.