themperek / cocotb-test

Unit testing for cocotb
BSD 2-Clause "Simplified" License
142 stars 70 forks source link

for Questa/Modelsim is mixedsvvh flag necessary for verilog compilation? #241

Open adambagley opened 1 year ago

adambagley commented 1 year ago

Hi, just wondering if the mixedsvvh flag could potentially be removed from this line: https://github.com/themperek/cocotb-test/blob/4924adc78712434dcd076060777239c04a22bfe8/cocotb_test/simulator.py#L578

Seems maybe unnecessary when compiling verilog code? It ends up causing lots of spamming warning messages in the simulator log file.

Thank you, Adam

manunited10 commented 4 months ago

Yeah gives error for me potentially for some complicated record of record type in a vhdl package (questasim)

** Error: (vcom-1609) The type used is not fully supported with the option -mixedsvvh.

Maybe we could make this mixedsvvh optional? It's not being generated when running cocotb through makefile. Thanks.