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.

Sign up for GitHub

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

Jump to bottom

Switch to main TVM repo in dependencies #47

Open
gussmith23 opened this issue Oct 9, 2020 · 1 comment
Open

Switch to main TVM repo in dependencies #47

gussmith23 opened this issue Oct 9, 2020 · 1 comment
Labels
good first issue Good for newcomers TVM integration Integration between the TVM deep learning compiler and Glenside

Comments

@gussmith23
Copy link
Owner

Currently pointing at mwillsey's fork, but now that his PR is merged, we can switch.
Should pick a specific revision from TVM, not just point at master!
@gussmith23 gussmith23 added good first issue Good for newcomers TVM integration Integration between the TVM deep learning compiler and Glenside labels Oct 9, 2020
@hypercubestart
Copy link
Collaborator

UPDATE: Once #94 is merged, TVM will point to hypercubestart's fork. Once Jared's Rust fixes go in (apache/tvm#7085) we should switch back to master!

@hypercubestart hypercubestart mentioned this issue Jan 20, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers TVM integration Integration between the TVM deep learning compiler and Glenside
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gussmith23 @hypercubestart