Skip to content
This repository was archived by the owner on Jun 3, 2024. It is now read-only.

Add ImportStructTypeOfPortWithFieldOfTheSameNameAsRangeParameter test #614

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

RRozak
Copy link
Member

@RRozak RRozak commented Dec 8, 2021

No description provided.

@RRozak RRozak requested a review from rkapuscik December 8, 2021 13:12
@@ -0,0 +1,12 @@
package lc_ctrl_pkg;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ [verible-verilog-lint] reported by reviewdog 🐶
Package declaration name must match the file name (ignoring optional "_pkg" file name suffix). declaration: "lc_ctrl_pkg" vs. basename(file): "top" [Style: file-names] [package-filename]

@RRozak
Copy link
Member Author

RRozak commented Dec 8, 2021

I created this test to check if there is no conflict between member name and parameter name.
I noticed that ranges of structs aren't parsed, so I created separate test for it: #615

@RRozak
Copy link
Member Author

RRozak commented Dec 8, 2021

It waits for chipsalliance/Surelog#2282

@RRozak
Copy link
Member Author

RRozak commented Jan 18, 2022

It still doesn't work

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

Successfully merging this pull request may close these issues.

2 participants