The-OpenROAD-Project / TritonMacroPlace

Macro placement tool for OpenROAD flow
BSD 3-Clause "New" or "Revised" License
21 stars 11 forks source link

Invalid floorplan for several OpenROAD designs #1

Closed tajayi closed 5 years ago

tajayi commented 5 years ago

Priority: High

I have observed several bad placement of macros in the OpenROAD flow.

overlapping macro placement bp_fe_top_tsmc65lp image

macro placement outside the core boxe swerv_wrapper_tsmc65lp image

maro sideways rotation (causes issues with power grid) tinyRocket_tsmc65lp image

Steps to Reproduce:

  1. Setup the latest version of the flow (commit e7abe2a8a010019f5a0c74472b96842cb66c2a71)
  2. Setup the tsmc65lp platform
  3. Attempt make floorplan on any of the designs listed above
mgwoo commented 5 years ago

The problem in tinyRocket_tsmc65lp and bp_fe_top_tsmc65lp is solved by the following commit: https://github.com/The-OpenROAD-Project/TritonMacroPlace/commit/f13ebbbf8f91d5f41f1d20d6bc780d632345a938