Open olofk opened 2 months ago
Both reasons can be solved by adding all the parameters to the top level/gate level modules and the error makes it obvious which parameters are missing.
I would not want to make this a warning in general, but we could possibly add an option to downgrade the error to a warning.
https://github.com/steveicarus/iverilog/commit/0ab1ed916fd3c6a3b55763fcaabcf00f18ceadf4 made it an error to apply non-existing parameters. While there are reasons for that, I would argue for reverting this change to make it a warning instead again. There are two semi valid reasons that I can cite.