-
Notifications
You must be signed in to change notification settings - Fork 489
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make Time + Delay column in Timing Report consistent with report_checks
when clocks are not expanded
#4916
Comments
You have ideal clocks enabled so the clock delay will be zero. It is showing you the computed delay but that value isn't summed in ideal clocks. We could zero it out though that info might be useful. |
I take that back - you do have clock propagation on. I don't understand the question though. In |
In the text report I see:
where the delay is 153.56 rather than 42.713. |
@maliberty Did you just answer your own question or is there a question for me? |
I don't see a bug here, perhaps a misunderstanding? |
Aha! I get it. Yes: I believe that this is a bug(or unintentional effect) in the representation in the GUI when lines are hidden. When clock lines are not hidden, I believe that we see the intended representation of the delay column. When clock lines are hidden, and one one wants the representation of the Delay column to be added to the previous Time to get current time, a choice has to be made for "clock network delay". It must either subtract the Delay on the next line or the Delay on the following line should be zero. I think it would be less surprising if the "ces_0_0/clock (Element)" line did not change, in which case, the "clock network delay" delay column have the delay of "ces_0_0/clock (Element)" subtracted. Also, the GUI should match Update GUI to match
|
report_checks
report_checks
report_checks
when clocks are not expanded
Description
For all rows in the Data Path Details, the Time column is the previous Time + the current Delay.
Example of what is expected: 484.724+41.142=525.866
However, if I do this for the
ces_0_0/clock (Element)
line, that is not the case. The 42.713 is not added to the previous line 331.162.This is from
make DESIGN_CONFIG=designs/asap7/mock-array/config.mk route
To reproduce, untar gui.tar.gz
Suggested Solution
Make the Delay column consistent with
report_checks
when clocks are not expanded:Additional Context
No response
The text was updated successfully, but these errors were encountered: