Skip to content

Make the counter synthesizable, update comment#1414

Draft
ShashankVM wants to merge 4 commits intodiffblue:mainfrom
ShashankVM:sv_syntax_fix
Draft

Make the counter synthesizable, update comment#1414
ShashankVM wants to merge 4 commits intodiffblue:mainfrom
ShashankVM:sv_syntax_fix

Conversation

@ShashankVM
Copy link
Copy Markdown
Contributor

  • Use nonblocking assignment for counter. Sequential design use nonblocking assignments.
  • Add reset and remove initial block as initial block is not synthesizable
  • Add counter output port
  • Update comment to count up

- Use nonblocking assignment for counter. Sequential design use nonblockgin assignments.
- Add reset and remove initial block as initial block is not synthesizable
- Add counter output port
@ShashankVM
Copy link
Copy Markdown
Contributor Author

Regression fails with error: unsupported property - only SVA s_eventually or AF implemented
Looks like the syntax is correct and not supported by the tool.

@ShashankVM ShashankVM marked this pull request as ready for review April 7, 2026 15:21
Copy link
Copy Markdown
Contributor Author

@ShashankVM ShashankVM left a comment

Choose a reason for hiding this comment

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

Regression test is failing. Code looks fine to me.

@ShashankVM ShashankVM marked this pull request as draft April 8, 2026 15:59
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

Successfully merging this pull request may close these issues.

1 participant