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

Format Module Instantiations #3

Open
alhassanalbadri opened this issue Apr 10, 2024 · 1 comment
Open

Format Module Instantiations #3

alhassanalbadri opened this issue Apr 10, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@alhassanalbadri
Copy link
Owner

The linter currently formats such module definitions with no issue:

module x(
input logic test1,
output logic test2,
);
logic f;
assign f = x;

endmodule

But it does not support module instantiations such as:

adder Add1and2(
.op1(test),
.op2(test),
.result(output)
)

We can probably detect these by the format they're created in: ModuleName instName( ...params )

@alhassanalbadri alhassanalbadri added the enhancement New feature or request label Apr 10, 2024
@alhassanalbadri
Copy link
Owner Author

With this commit: 71ed26d modules inst. are now detected but are not formatted as of yet.

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

No branches or pull requests

1 participant