Skip to content
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

Linting with Verilator 5: SystemVerilog library files incorrectly processed as Verilog files #45

Open
aumouvantsillage opened this issue Aug 28, 2023 · 3 comments

Comments

@aumouvantsillage
Copy link

I am using svlangserver in Neovim with the default settings.
The linter is Verilator 5.015.

In my projects, the linter emits this error message for all imports:

Importing from missing package 'P'

However, the corresponding package files are all listed correctly in the linter.vc file generated by svlangserver.

I have noticed that the errors disappear if I edit the file linter.vc and I remove the -v switch before the library file names.
In fact, according the Verilator man page, this option is documented as:

-v Verilog library

I suspect that Verilator 5 does not support SystemVerilog for files introduced with the -v option.
Just listing the file names alone seems to work.

@kkanhere
Copy link
Collaborator

kkanhere commented Sep 5, 2023

Thanks for filing an issue.Verilator linting is broken at the moment and I plan to fix it in the next release.

@jagjordi
Copy link

Is there any plan of fixing this? What would be needed? I could give it a try!

@kkanhere
Copy link
Collaborator

It is definitely in the TODO list.
An immediate simple fix would be to just revert the logic that was changed around linting in this release as the previous release had better linting functionality.
In the long run the idea should be to create a vc file per file being edited with precisely its dependencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants