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

Change local to const #909

Closed
dalance opened this issue Aug 23, 2024 · 0 comments · Fixed by #932
Closed

Change local to const #909

dalance opened this issue Aug 23, 2024 · 0 comments · Fixed by #932
Labels
breaking-change lang Language design

Comments

@dalance
Copy link
Collaborator

dalance commented Aug 23, 2024

local is derived from localparam of SV. But the meaning is confusable, especially non-SV users.
local looks "local scoped value", but actually local of package can be seen globally.
In general programming languages, const is used for this usage.

Additionally in #866, const will be used as generic parameter bound. So I think changing local to const at the same time enhances language consistency.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
breaking-change lang Language design
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change local to const veryl-lang/veryl
1 participant
@dalance