mpl: support timing-driven macro placement#9249
mpl: support timing-driven macro placement#9249AcKoucher wants to merge 3 commits intoThe-OpenROAD-Project:masterfrom
Conversation
Signed-off-by: Arthur Koucher <arthurkoucher@precisioninno.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces timing-driven macro placement by integrating the Resizer module to identify critical nets. The changes consistently extend the existing simulated annealing algorithm to incorporate critical wire length into its cost function. The modifications span across various components, including the MacroPlacer initialization, CMakeLists.txt for linking, header files for new methods and members, and source files for implementation details. The Tcl interface has also been updated to expose the new timing_driven option. The changes are well-contained and appear to be functionally sound.
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
|
||
| // Variables for data flow | ||
| DataFlow data_connections_; | ||
| sta::NetSeq critical_nets_; |
There was a problem hiding this comment.
warning: no header providing "sta::NetSeq" is directly included [misc-include-cleaner]
src/mpl/src/clusterEngine.h:21:
+ #include "sta/NetworkClass.hh"
Signed-off-by: Matt Liberty <mliberty@precisioninno.com>
2 participants
Draft to present the general idea.