-
Notifications
You must be signed in to change notification settings - Fork 10
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: Snitch Cluster Tiling Support #22
OPEN: Snitch Cluster Tiling Support #22
Conversation
extend snitch support
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.
First round of comments, I haven't had time to give a pass on everything yet.
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.
All my comments from my previous pass were addressed. I have two more comments that require some changes, mostly refactoring, since you know the codebase by now this should be done in a day :)
f0d7c69
to
6fe7c42
Compare
LGTM, ready to merge as soon as the pipeline passes. |
This PR mirrors a PR from the GitLab repository, extending the support for
snitch_cluster
platform by adding support for single-buffered tiling and more operators, such as:Added
ExecutionBlock
.ExecutionBlock
.--debug
flag tocargo install
when installing Banshee to get the possibility of enabling the debug prints.snitch_cluster
platform.main.c
to disable printing and testing (convenient when running RTL simulations).Changed
Fixed
PR Merge Checklist
devel
commit and pointing todevel
.CHANGELOG.md
file has been updated.