Testbench does not verilate

chipsalliance / Cores-VeeR-EH1

VeeR EH1 core

Apache License 2.0

808 stars 219 forks source link

Testbench does not verilate #13

Closed toddstrader closed 5 years ago

toddstrader commented 5 years ago

The Verilator external test suite is breaking on the SweRV test, and this time I don't believe it is a Verilator regression: https://travis-ci.com/verilator/verilator_ext_tests/jobs/228899219

It appears that commit 0dacc978 moved finished outside of a macro block that was probably there to get around this issue in the first place. Now Verilator complains that it is defined twice.

aprnath commented 5 years ago

Thanks for reporting this, Todd. I have pushed a patch for this # fc331027c

toddstrader commented 5 years ago

Confirmed, thanks: https://travis-ci.com/verilator/verilator_ext_tests/builds/125826326?utm_medium=notification&utm_source=email