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.

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

[enhancement] Name legalization should choose keywords based on configuration #1641

Open
grebe opened this issue Oct 2, 2024 · 0 comments
Open
Labels
codegen Related to emitting (System)Verilog. enhancement New feature or request

Comments

@grebe
Copy link
Collaborator

grebe commented Oct 2, 2024

What's hard to do? (limit 100 words)

You might want to do any of the following:

  • Generate Verilog but avoid all SystemVerilog keywords b/c the file will be used in multiple contexts
  • Generate SystemVerilog that avoids keywords in other standards, e.g. Verilog-AMS
  • Choose specific versions of standards that need to allow future keywords for compatibility reasons

Name legalization currently chooses its set of keywords entirely based on what kind of RTL is being generated, so these things aren't possible.

Current best alternative workaround (limit 100 words)

Hack name legalization to do what you want or post-process the RTL.

Your view of the "best case XLS enhancement" (limit 100 words)

A separate codegen flag specifying which standards' keywords to avoid.

@grebe grebe added enhancement New feature or request codegen Related to emitting (System)Verilog. labels Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
codegen Related to emitting (System)Verilog. enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant