OpenROAD icon indicating copy to clipboard operation
OpenROAD copied to clipboard

CTS-Avoid level balance buffers overlap.

Open arthurjolo opened this issue 1 year ago • 6 comments

The current Level Balancer of CTS tends to insert buffers one on top of the other and when they are legalized a cluster of buffers is formed, issue #2305 shows an example of this. This happens specially if the clock buffers are large and it creates a high congested area that can lead to the global router finishing with overflow.

This PR changes the level balancer to avoid overlapping the inserted buffers and preventing the creation of the clusters of buffers. The next images show the result for the regressions test of CTS balance_levels: Current: image

New: image

arthurjolo avatar Sep 27 '24 22:09 arthurjolo

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Sep 27 '24 22:09 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Sep 30 '24 18:09 github-actions[bot]

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Oct 04 '24 16:10 github-actions[bot]

secure ci?

maliberty avatar Oct 04 '24 17:10 maliberty

Secure Ci is passing, only need to update a antenna diode count metric for 1 design. I am going to create a PR for this update.

arthurjolo avatar Oct 04 '24 17:10 arthurjolo

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Oct 04 '24 19:10 github-actions[bot]

Close as balance levels is now obsolete.

arthurjolo avatar Oct 02 '25 17:10 arthurjolo