Olof Kindgren

Aarrrghh!!! Seems like there's some cmake scripts that needs updating. I have no idea what I'm doing anymore

Ok, fuck it. I give up with the cmake stuff and simplified the PR instead so that it doesn't touch symbiflow_synth at all. This can't possibly go wrong

Yes! All green. Good to go now @acomodi ?

@acomodi Rebased and repushed

The core description file looks great to me. There are quite a few FuseSoC VHDL users so it would be great to have proper support for OSVVM

This looks great and will be very useful to keep up code quality. Hopefully we cam reduce the number of existing warnings too. There are quite a few of them...

I see two benefits with this patch. 1. Currently there are so many warnings so that there is no choice but to turn them off, which is always a bad...

This would be very good to get fixed. I have now added FuseSoC support to run the caravel regression tests that use this library, but it only works on icarus....

I can confirm that this works fine with QuestaSim and Xilinx XSim after these changes. If dropping default_nettype none is too controversial, I have already started the work to explicitly...

Ouch, you're right. I didn't consider the generated netlists :/ I guess this should be reported to Yosys as well for the future, or maybe that is already done?