Support for FST traces in verilator backend#250
Support for FST traces in verilator backend#250kammoh wants to merge 1 commit intofreechipsproject:masterfrom
Conversation
chick
left a comment
There was a problem hiding this comment.
Thanks for the PR. This looks pretty good. I have a couple of small problems
First: Can you please make some kind of test that demonstrates the use of the flag and that
the .fst file was created. GcdExampleSpec, for instance, test whether a .vcd was created.
Second: There are quite a few changed lines in this PR that appear to be cosmetic, different spacing.
It makes it harder for the reviewer to focus on the core change.
If you'd like to reformat things to be better style compliant that's great, but best to keep that in a separate style PR.
|
@ekiwi I know you have some interest in .fst format, or is your use case just for treadle |
|
Thanks for adding me @chick. Do you know if the verilator version is available in the backend? As far as I remember the |
Introducing the ability to use FST format as the output format of
Verilatortrace dump, instead of the VCD format. FST is a more efficient trace format introduced by thegtkwaveproject.A boolean option named
verilatorFstTraceis currently added toTesterOptions, but can be eventually ported intochisel-testers2with an appropriate Annotation.