search

The-OpenROAD-Project / OpenSTA

OpenSTA engine
GNU General Public License v3.0
415 stars 174 forks source link

Found might be functionality inconsistency in Search::seedClkArrival with Search::thruClkInfo #221

Closed icbenchpeng closed 3 months ago

icbenchpeng commented 9 months ago

The inconsistency is about sdc_->isPropagatedClock(pin).

seedClkArrival do always check sdc->isPropagatedClock if pin clock latency does not exists. thruClkInfo first check sdc->isPropagatedClock of the local pin, but then overwrites when pin's clock latency does not exist but hierarchical pin latency thru edge exist. Is there an assumption exist here ?

jjcherry56 commented 7 months ago

I can't parse your description to understand what your point is. Exactly what assumption are you speculating exists? If you can create a test case that demonstrates a bug I can look into it.

maliberty commented 3 months ago

Issues or PRs should be filed with https://github.com/parallaxsw/OpenSTA if still relevant. This is effectively a fork (though not strictly for historical reasons).