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

Netify #603

Open
wants to merge 13 commits into
base: master
Choose a base branch
from
Open

Netify #603

wants to merge 13 commits into from

Conversation

rdaly525
Copy link
Owner

@rdaly525 rdaly525 commented Aug 21, 2018

This pass "netifies" the graph. What this means is that every connection from an instance always connects to a "net" (a coreir.wire instance). This allows a possible verilog implementation where the verilog can have less wire declarations. This is also possibly useful for timing analysys, and is the structure for other serialization formats.

@rdaly525 rdaly525 requested a review from leonardt August 21, 2018 14:55
@leonardt
Copy link
Collaborator

@rdaly525 this what out of sync with dev, I tried to resolve the conflicts with b4487f8, can you review them to make sure they make sense?

@leonardt
Copy link
Collaborator

It maybe also be useful to review the new changeset at https://github.com/rdaly525/coreir/pull/603/files to see if they include what you wanted (sorry this is a bit old, hopefully you remember).

@rdaly525
Copy link
Owner Author

rdaly525 commented Feb 8, 2019

Resynced this with the current dev branch. This pass should be useful for @makaimann

@rdaly525 rdaly525 changed the base branch from dev to master January 20, 2021 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants