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

Propagate type annotations in the exact matcher #22

Open
wants to merge 133 commits into
base: 3la-pldi-push-main
Choose a base branch
from

Conversation

slyubomirsky
Copy link
Collaborator

This change adds a small but potentially useful feature to the exact matcher: If the checked types have already been filled in for the target program, this will add in type annotations to the inserted functions. This should address the bug we ran into with type checking on LSTM (don't know the root cause on that).

Please review @AD1024 @gussmith23

AD1024 and others added 30 commits May 24, 2021 05:14
* [ add ] vat_matmul runtime files

* [ update ] get rid of pushing 0s to ACC

* add comment about the change in codegen.cc

* restore ACC buffer initialization

* [ add ] compute missing fields of VTA config
@gussmith23
Copy link
Collaborator

@slyubomirsky can you check that my merge conflict resolution is correct? and then can you merge this?

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.

6 participants