-
Notifications
You must be signed in to change notification settings - Fork 621
Couple of changes to get this running on ubuntu 2204 with iverilog 11 #13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Couple of changes to get this running on ubuntu 2204 with iverilog 11 #13
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
reg
from 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