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

[FIRRTL] Remove lowerToBind from the FIRRTL Instance Op #8386

Open
rwy7 opened this issue Apr 2, 2025 · 0 comments
Open

[FIRRTL] Remove lowerToBind from the FIRRTL Instance Op #8386

rwy7 opened this issue Apr 2, 2025 · 0 comments
Assignees
@rwy7
Labels
FIRRTL Involving the `firrtl` dialect

Comments

@rwy7
Copy link
Contributor

rwy7 commented Apr 2, 2025

Once we have bind ops in the FIRRTL dialect, we can drop the lowerToBind flag for FIRRTL instance ops. Users will have to create their own bind ops, rather than rely on lowerToHW to do it for them.
@rwy7 rwy7 added the FIRRTL Involving the `firrtl` dialect label Apr 2, 2025
@rwy7 rwy7 self-assigned this Apr 2, 2025
@rwy7 rwy7 mentioned this issue Apr 2, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rwy7 rwy7

Labels
FIRRTL Involving the `firrtl` dialect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rwy7