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] IR Conversion could have a default FIFO config #1630

Closed
grebe opened this issue Sep 25, 2024 · 0 comments
Closed

[enhancement] IR Conversion could have a default FIFO config #1630

grebe opened this issue Sep 25, 2024 · 0 comments
Labels
dslx DSLX (domain specific language) implementation / front-end enhancement New feature or request 🧦 sox

Comments

@grebe
Copy link
Collaborator

grebe commented Sep 25, 2024

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

Currently, the ways to specify FIFO configuration are limited and tedious (see #1561 as wells as #1391 + #989). Furthermore, because FIFO configurations are only relevant at codegen, you can write a bunch of DSLX, test it, and then have to go back and update every chandecl to get it to codegen.

Current best alternative workaround (limit 100 words)

Manually specify DEPTH everywhere until things build.

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

In lieu of enhancements in #1561 making FIFO specification easier and better, it is easy to implement and helpful to have an IR conversion option where you specify a default FIFO configuration. It could be something like --default_fifo_config=<textproto of FifoConfig>

@grebe grebe added enhancement New feature or request dslx DSLX (domain specific language) implementation / front-end 🧦 sox labels Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dslx DSLX (domain specific language) implementation / front-end enhancement New feature or request 🧦 sox
Projects
Status: Done
Development

No branches or pull requests

1 participant