search

The-OpenROAD-Project / OpenROAD-flow-scripts

OpenROAD's scripts implementing an RTL-to-GDS Flow. Documentation at https://openroad-flow-scripts.readthedocs.io/en/latest/
https://theopenroadproject.org/
Other
282 stars 262 forks source link

macro placement: less surprising logging #2069

Closed oharboe closed 1 week ago

oharboe commented 1 week ago

The [INFO][FLOW-xxxx] prefix is surprising, leaves questions... What is xxxx? Is something wrong? Should the code be updated?

Generally the .tcl code just puts logging information, so do the same here. There are no other occurrences in the .tcl scripts of [INFO][FOO-].

The openroad binary has error/warning/info numbers that are allocated and guaranteed by build time checks to be unique.

rovinski commented 1 week ago

xxxx should have been a 4 digit number just like OR messages. But there is no enforcement mechanism to do so, so things like this slip through the cracks.