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

Add ParameterWithOverriddenWidthInitializedByConst test #577

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

Add ParameterWithOverriddenWidthInitializedByConst test #577

wants to merge 1 commit into from

Conversation

RRozak
Copy link
Collaborator

@RRozak RRozak commented Oct 25, 2021

No description provided.

@RRozak RRozak mentioned this pull request Oct 25, 2021
Closed
@RRozak
Copy link
Collaborator Author

RRozak commented Oct 27, 2021

It waits for chipsalliance/Surelog#2141

@RRozak
Copy link
Collaborator Author

RRozak commented Nov 19, 2021

It still doesn't work.

@RRozak
Copy link
Collaborator Author

RRozak commented Dec 8, 2021

It doesn't work due to warning: ParameterWithOverriddenWidthInitializedByConst//top.sv:9:45: Value too large for 1 bit number: 15.
It is thrown during reading uhdmAllModules section in which the parameters have default values.
It doesn't have any impact on simulation. I will leave this test for now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kbieganski kbieganski kbieganski approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@RRozak @kbieganski