djg / verilated-rs

Verilator Porcelain
Apache License 2.0
37 stars 15 forks source link

verilator v4.038 flush changes. #6

Closed djg closed 3 years ago

djg commented 3 years ago

The interface to running callbacks on flush changes in Verilator v4.038. Add support to check version of verilator being used and compile in the the correct methods in the shim.

Going forward, verilated-rs will drop support for Verilator versions before v4.108

djg commented 3 years ago

@tommythorn Are you able to get the example working with Verilator v4.038 with this PR?

tommythorn commented 3 years ago

I’ll test

tommythorn commented 3 years ago

The flush_and_exit_cb branch builds for me with v4.038 (tested on Ubuntu) and 4.108 (tested on macOS + HomeBrew), thanks. On both cargo run -p example works correctly.

Thanks

djg commented 3 years ago

Fixes #5