Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

opt_clean, simplemap: Add $buf handling #4931

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

povik
Copy link
Member

@povik povik commented Mar 7, 2025

What are the reasons/motivation for this change?

The write_xaiger2 command transiently enters the buffered-normalized mode. This can introduce $buf cells into the design. These cells are missing handling by the opt_clean and techmap passes, and as such they just fall through to the end of a flow.

So far the bufnormed mode usage has been isolated to the insides of a single pass, once we will want to extend this to let designs in bufnormed mode cross from one pass to another, we will need to revisit the opt_clean/techmap behavior. At that point we will need to revisit all other cases where these passes introduce connections anyway.

If applicable, please suggest to reviewers how they can test the change.

There's a minimal buf.ys test of the new behavior

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant