Couple of changes to get this running on ubuntu 2204 with iverilog 11#13
Couple of changes to get this running on ubuntu 2204 with iverilog 11#13niyas-sait wants to merge 3 commits intoadam-maj:masterfrom
Conversation
|
Have you tried enabling the systemverilog flag (-g2012) with this command?
I believe Icarus Verilog runs the 2005 version (-g2005) by default. |
|
Realised this other PR uses the updated flag as well, think it might just be the fix: https://github.com/adam-maj/tiny-gpu/pull/2/files |
|
I think you're right that that's the fix for everyone @adviyer - will push that when I'm back at my computer |
|
@niyas-sait if you want to remove all the reg changes in this PR and just update to use the 2012 version I can merge this or alternatively will just make the change myself |
|
I tried top of master (9a07e2b) and still getting lots of errors during compilation. |
Getting bunch of errors during compilation due to declaring input as reg type.
Dropping
regfrom input seems to fix this issue. I am not a Verilog expert, so not sure if this is specific to the iverilog (version 11) compiler I am using or the problem with SV2V conversion.PR also contains some minor updates to README and Makefile