Skip to content

Verilog: parse fork...join_any and fork...join_none - #1950

Open
kroening wants to merge 1 commit into
diffblue:mainfrom
kroening:fix-fork-join-any-none
Open

Verilog: parse fork...join_any and fork...join_none#1950
kroening wants to merge 1 commit into
diffblue:mainfrom
kroening:fix-fork-join-any-none

Conversation

@kroening

@kroening kroening commented Jun 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

The parser only handled fork...join. Both join_any and join_none are standard SV parallel block terminators (IEEE 1800-2017 9.3.2). Add grammar rules for the named and unnamed variants of each, mapping them to the same ID_fork/ID_block representation as join.

🤖 Generated with Claude Code

@kroening
kroening force-pushed the fix-fork-join-any-none branch from db193a9 to 80675ed Compare June 28, 2026 18:34
@kroening
kroening marked this pull request as draft June 28, 2026 19:33
@kroening kroening changed the title verilog: parse fork...join_any and fork...join_none Verilog: parse fork...join_any and fork...join_none Jun 28, 2026
@kroening
kroening force-pushed the fix-fork-join-any-none branch from 80675ed to e9f9a69 Compare August 9, 2026 14:56
@kroening
kroening marked this pull request as ready for review August 9, 2026 14:57
@tautschnig

Copy link
Copy Markdown
Collaborator

Tests, please.

The parser only handled fork...join. Both join_any and join_none are
standard SV parallel block terminators (1800-2017 9.3.2). Map them to
the same ID_fork/ID_block representation as join for now.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kroening
kroening force-pushed the fix-fork-join-any-none branch from e9f9a69 to 64a3c9a Compare September 5, 2026 23:38
@kroening

kroening commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator Author

Now with tests.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants