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

'Goto definition' doesn't work if there is a package import in the module header #189

Open
lvoudour opened this issue Dec 7, 2022 · 1 comment

Comments

@lvoudour
Copy link

lvoudour commented Dec 7, 2022

If a module has a package import declaration in the module header, then when the module is instantiated 'Goto definition' and hover don't work on the module name

Module header:

module foo
  import foo_pkg::*; 
#(parameter z = 1) (
  input  logic x,
  output logic y
);

Module instantiation:

// Cannot ctrl+click 'foo' or see hover information
foo #(.z(3)) u_foo (
  .x(x),
  .y(y)
);

In fact, it seems that even a comment between the module name and the parameter list declaration can throw the parser off:

module foo //this comment breaks the parser
#( parameter z = 1) (
  input  logic x,
  output logic y
);
@wynnie
Copy link

wynnie commented Jul 19, 2023

Running into the same issue.

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

2 participants