search

The-OpenROAD-Project / OpenROAD

OpenROAD's unified application implementing an RTL-to-GDS Flow. Documentation at https://openroad.readthedocs.io/en/latest/
https://theopenroadproject.org/
BSD 3-Clause "New" or "Revised" License
1.35k stars 481 forks source link

mpl2: decrease connection signature weight threshold #5017

Closed AcKoucher closed 2 weeks ago

AcKoucher commented 2 weeks ago

When running Secure-CI for #4991 asap7/aes-block showed some metrics degradation. After some digging it looked like the problem wasn't specifically related to that PR itself, but rather there was something weird going on with the clustering.

Actually, asap7/aes-block shows a very narrow case where the minimum weight threshold for MPL2 to consider two clusters connected was impacting on how the clusters were formed.

These changes are to ensure that, for smaller designs, we won't end up splitting macros unconveniently.

github-actions[bot] commented 2 weeks ago

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

AcKoucher commented 2 weeks ago

Running Secure-ci