chipsalliance / chipsalliance/chisel

Line up poke() and register output changes in .vcd trace

Open
#5,107 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Scala
Stars
4.8k
Forks
658
Avg merge
18h 59m
Merged PRs (30d)
14

Description

**Type of issue**: Feature Request

**Is your feature request related to a problem? Please describe.**

Today poke() signals are shown as changing on the falling edge.

I would like to see poke input signals immediately after the rising edge of the clock but before the waveform dump for that timestep occurs. This makes the .vcd dumps easier on the eye, because the change due to a peek/poke lines up with the change of a register.

**Describe the solution you'd like**

Image

Showing poke() signals changing on the rising edge of a clock in Verilator dumps can be achieved in the generated C++ harness by inserting a poke/eval phase between the clock rise and the dump. The desired execution flow is:

```cpp
clock = 0;
timeInc(1);
eval();
dump();

clock = 1;
timeInc(1);
eval(); // Logic settles on rising edge

// Pokes happen here

if (trace) {
// Re-evaluate to propagate pokes immediately
eval();
}
// changes from output of registers and poke() now appear on the rising edge
dump();

```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.