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

Cts prevent removing sinks #5207

Closed arthurjolo closed 2 weeks ago

arthurjolo commented 2 weeks ago

During the HTree building step of cts we can create more Clock subNets then needed, these extra Clock subNets were deleted at the end of CTS, but this can cause crashes (like it was mentioned in PR#4938).

To prevent having to delete these nets at the end of CTS, this PR identifies and removes theses nets during the HTree building step.

github-actions[bot] commented 2 weeks ago

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

arthurjolo commented 2 weeks ago

This PR generated a crash in one testcases, and the approach to solve the problem had to change. So I will be closing this PR and open a new one with the correct fix.