-
Notifications
You must be signed in to change notification settings - Fork 46
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
Complete AXI wrapper generator #1934
Merged
Merged
Changes from all commits
Commits
Show all changes
30 commits
Select commit
Hold shift + click to select a range
d3ac4c6
WIP axi-gen main component
nathanielnrn 7498f38
Add clearer exceptions in invokes
nathanielnrn d12071f
get invoke of channels working
nathanielnrn e8cb4ba
Add `*=` for par block control composition
nathanielnrn 7262a54
Add small runt test for `+=` and `*=`
nathanielnrn f558c28
add some documentation about *=
nathanielnrn f6fd273
Merge branch 'main' into builder-qol
nathanielnrn b507509
better exceptions py builder
nathanielnrn 7a28be6
Merge branch 'main' into py-axi-gen-main
nathanielnrn a9628ce
Merge branch 'builder-qol' into py-axi-gen-main
nathanielnrn c626bc0
AXI generator that creates a valid calyx program
nathanielnrn eb3e451
Revert "Merge branch 'builder-qol' into py-axi-gen-main"
nathanielnrn ebec96c
Add invocation and instantiaion of main_compute
nathanielnrn 75dafbb
Attempt at generalizing attributes
nathanielnrn e043f59
Add component attribute support to builder
nathanielnrn 3e0a741
update invoke.py in test/
nathanielnrn 0aadcfa
typo fix in docs
nathanielnrn 2763154
Merge branch 'builder-attributes' into py-axi-gen-main
nathanielnrn 5431d97
Add toplevel attribute to `wrapper`
nathanielnrn fcfa0c8
Make fake yxi align with vec-add memory widths
nathanielnrn 9f624ce
Cleanup and add dynamic-wrapped-vec-add
nathanielnrn 806b985
Get generated wrapper to run properly
nathanielnrn f089ddd
Add working generated wrapper + vec_add
nathanielnrn 78078a4
Merge branch 'main' into py-axi-gen-main
nathanielnrn 50516ec
remove redundant attributes argument in py_ast
nathanielnrn 5419b3d
Add space to py_ast.py to pass runt tests
nathanielnrn 10b4ef2
rename read_en to content_en
nathanielnrn 09f1dc9
Merge branch 'py-axi-gen-main' of github.com:calyxir/calyx into py-ax…
nathanielnrn 883cac8
add runt test on generation of calyx-py axi wrapper
nathanielnrn 05743f1
Merge branch 'main' into py-axi-gen-main
nathanielnrn File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good for now! In the future, you can imagine this using a set of YXI inputs as the test files, so we see different AXI-wrapper code generated for each one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I mentioned this in #1928, which is not very fleshed out currently, but I think(?) points to what you're getting at.