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

Add UnpackedUnionArray test #683

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

RRozak
Copy link
Member

@RRozak RRozak commented Feb 24, 2022

No description provided.

Signed-off-by: Ryszard Różak <[email protected]>
} my_union_t;

initial begin
my_union_t un [9:0];

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 🐶
Unpacked dimension range must be declared in big-endian ([0:N-1]) order. Declare zero-based big-endian unpacked dimensions sized as [N]. [Style: unpacked-ordering] [unpacked-dimensions-range-ordering]

Signed-off-by: Ryszard Różak <[email protected]>
@RRozak
Copy link
Member Author

RRozak commented Mar 3, 2022

Unpacked unions are unsupported by both verilators and both yosys. It's better to wait with the merge until there will be at least one tool that can be tested on it

@@ -0,0 +1,12 @@
module top1(output logic [3:0] o);

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 🐶
Declared module does not match the first dot-delimited component of file name: "top" [Style: file-names] [module-filename]

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.

1 participant