dpl: don't touch fixed cells inside groups

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.41k stars 494 forks source link

dpl: don't touch fixed cells inside groups #5209

Closed eder-matheus closed 2 weeks ago

eder-matheus commented 2 weeks ago

Fixes https://github.com/The-OpenROAD-Project/OpenROAD/issues/5204

github-actions[bot] commented 2 weeks ago

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

eder-matheus commented 2 weeks ago

Looks good, do you see any other places that need the same?

This was the only place with the missing check in DPL. I inspected the code for possible causes of the issue, and the other places already checked if the Cell is a fixed inst.