Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Enhance case statement/expression #783

Merged
merged 3 commits into from
Jun 14, 2024
Merged

Enhance case statement/expression #783

merged 3 commits into from
Jun 14, 2024

Conversation

taichi-ishitani
Copy link
Contributor

@taichi-ishitani taichi-ishitani commented Jun 12, 2024 •
edited
Loading

  • Case statement/expressoin
    • use case inside/==? operator
    • support range expression on case condition
    • introduce limitation that case condition should be constant
      • this is to prevent mismatch between simulation and synthesis
  • Swich statement/expression
    • this is for idiom of onehot csae statement
      • case (1'b1) ... endcase
    • expression can be uesd for switch condition

resolve #643

@taichi-ishitani taichi-ishitani force-pushed the enhance_case_statement_expression branch from e2819a5 to 6279cfa Compare June 12, 2024 07:54
@taichi-ishitani taichi-ishitani force-pushed the enhance_case_statement_expression branch from 6279cfa to b5a72c5 Compare June 13, 2024 05:23
@taichi-ishitani taichi-ishitani marked this pull request as ready for review June 13, 2024 05:28
@dalance dalance added the enhancement New feature or request label Jun 14, 2024
@dalance
Copy link
Collaborator

dalance commented Jun 14, 2024

Looks good!
I'll merge this PR.

@dalance dalance merged commit 7c483e6 into veryl-lang:master Jun 14, 2024
6 checks passed
@taichi-ishitani taichi-ishitani deleted the enhance_case_statement_expression branch June 14, 2024 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Feature] case inside statement and expression
2 participants
@taichi-ishitani @dalance