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] Put module_builder's functions in a shared package #1643

Open
grebe opened this issue Oct 3, 2024 · 0 comments
Open

[enhancement] Put module_builder's functions in a shared package #1643

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

Comments

@grebe
Copy link
Collaborator

grebe commented Oct 3, 2024

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

Currently, module_builder puts functions in each module. With multi-proc codegen, this means you end up redefining a bunch of functions in many cases.

Current best alternative workaround (limit 100 words)

Accept the current behavior- it works.

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

At least for SystemVeriog, it might be nice to pull all these functions outside of their modules and put it in a package instead.

@grebe grebe added enhancement New feature or request codegen Related to emitting (System)Verilog. labels Oct 3, 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