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.38k stars 485 forks source link

DRT: disable init gc surgicalFix #5092

Closed openroad-robot closed 2 weeks ago

openroad-robot commented 2 weeks ago

When a worker has only recheck markers, the worker starts by initializing the route queue based on an initial gc check. If the gc check returns no markers, the worker is skipped because there is nothing to do. When surgicalFix is enabled at the gc worker, it patches minstep and minarea violations and removes them from the returned result. If we combine the 2 scenarios, we end up with a worker that does some patching at the gc level, returns no violation and does not update the final routing. This ends up in unrecognized minarea and minstep violations.

github-actions[bot] commented 2 weeks ago

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

maliberty commented 2 weeks ago

I restarted pr-head as it looks to have timed out