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

Genesis improvements #20

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

Genesis improvements #20

wants to merge 5 commits into from

Conversation

rsetaluri
Copy link
Collaborator

-- Capture entire genesis output
-- Cache genesis generators

Removes need to do any caching logic in downstream tools (gemstone does it for you). Also don't need to do any hacky copying of files etc. It is all captured in DeclareFromVerilogFile used in (run_genesis) now.

@Kuree
Copy link
Member

Kuree commented May 2, 2019

Can you make a PR in garnet that reflect this change?

@Kuree
Copy link
Member

Kuree commented May 2, 2019

Also how is it related to phanrahan/magma#367? I didn't see any changes here reflecting the magma change.

@Kuree
Copy link
Member

Kuree commented May 3, 2019

@rsetaluri Is this PR still relevant?

@rsetaluri
Copy link
Collaborator Author

Yes, what we did was just to revert to the previous verilog wrapping flow. I will work on getting this working on the side.

rsetaluri added a commit to StanfordAHA/garnet that referenced this pull request May 7, 2019
Updated memory core genesis wrapper test for new verilog/genesis
wrapping methodology.

Depends on StanfordAHA/gemstone#20.
@rsetaluri rsetaluri force-pushed the genesis-improvements branch from 3e77edf to 470d3e9 Compare May 7, 2019 22:35
@Kuree
Copy link
Member

Kuree commented May 13, 2019

@rsetaluri What is the status on this?

@Kuree Kuree force-pushed the master branch 4 times, most recently from 3ed5dc7 to 2b36a7e Compare November 18, 2019 19:14
@Kuree Kuree force-pushed the master branch 2 times, most recently from 6b36684 to 88a6bac Compare May 24, 2022 18:33
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.

2 participants