You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In chisel-testers, the poke-step-expect sequence works as expected, but since simif's "step" is really "fire one cycle of targetFire", the expect line actually dequeues the old value of io_out right before the posedge, which is different from what the chisel-testers do.
Suggestions: either document this, or perhaps rename as "targetFireStep()" or some other disambiguated name, or provide step(1) as an alias for targetFireStep(2), etc.
step()
in simif can be deceptive for users who are familiar with Chisel's PeekPokeTesters. Consider the following example:In chisel-testers, the poke-step-expect sequence works as expected, but since simif's "step" is really "fire one cycle of targetFire", the expect line actually dequeues the old value of io_out right before the posedge, which is different from what the chisel-testers do.
Suggestions: either document this, or perhaps rename as "targetFireStep()" or some other disambiguated name, or provide
step(1)
as an alias fortargetFireStep(2)
, etc.@davidbiancolin
The text was updated successfully, but these errors were encountered: