Skip to content

[OM] Remove Map and Tuple support. #8324

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

Open
mikeurbach opened this issue Mar 17, 2025 · 0 comments
Open

[OM] Remove Map and Tuple support. #8324

mikeurbach opened this issue Mar 17, 2025 · 0 comments
Labels
enhancement New feature or request OM Object Model

Comments

@mikeurbach
Copy link
Contributor

We have long deprecated and removed these from FIRRTL, so unless you emit OM dialect directly from a different frontend, there is no way to make use of these. And if you do have a frontend that emits the OM dialect directly, I would highly encourage you to instead make sure of Class, Object, and List to achieve the same goals. This is what Chisel does.

So, given these are redundant, mostly dead code, we should clean them up. I started on this long ago in a branch here: mikeurbach/om-map-cleanup. I had to pause because I ended up finding some internal users of these features.

@mikeurbach mikeurbach added enhancement New feature or request OM Object Model labels Mar 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request OM Object Model
Projects
None yet
Development

No branches or pull requests

1 participant