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

[CI] Include static builds of the runtime as part of CI #13612

Merged
merged 1 commit into from
Feb 8, 2023

Conversation

gigiblender
Copy link
Contributor

This PR partially addresses #13526 by adding static builds of tvm_runtime.a on Windows and Linux.

Tested at https://github.com/gigiblender/tvm/actions/runs/3694697105

@tvm-bot
Copy link
Collaborator

tvm-bot commented Dec 14, 2022

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

@gigiblender gigiblender force-pushed the build-static-runtime branch from 5ba8295 to 4f4c7d2 Compare January 8, 2023 18:15
@mehrdadh
Copy link
Member

mehrdadh commented Feb 1, 2023

@gigiblender I think this PR needs to be rebased

@gigiblender
Copy link
Contributor Author

@gigiblender I think this PR needs to be rebased

@mehrdadh ready for review

@mehrdadh mehrdadh requested a review from driazati February 7, 2023 21:50
@mehrdadh
Copy link
Member

mehrdadh commented Feb 7, 2023

Thanks @gigiblender, LGTM!
cc @driazati for review as well

Copy link
Member

@driazati driazati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@driazati driazati merged commit 6fb961a into apache:main Feb 8, 2023
driazati added a commit to driazati/tvm that referenced this pull request Feb 10, 2023
This was added in apache#13612 but is failing intermittently on `main` and
PRs (apache#13950), so this PR disables it until we can come up with a
consistent fix
driazati added a commit that referenced this pull request Feb 10, 2023
This was added in #13612 but is failing intermittently on `main` and
PRs (#13950), so this PR disables it until we can come up with a
consistent fix
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.

4 participants