[CI] Cross-compile libtvm_runtime to Aarch64 and run tests #13714
Conversation
|
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 is this PR related to why every other PR has suddenly failed the build? |
@Mousius I am not sure I see a correlation --- Could you please share the build error? I think the Jenkinsfile in this PR should not be picked up by Jenkins yet. |
|
@gigiblender I just noticed that the minimal build appeared in a bunch of PRs and has failed without any Jenkins link - example #13706 |
gigiblender
force-pushed
the
cross-build-arm
branch
3 times, most recently
from
07ac816 to
e674777
Compare
gigiblender
changed the title
There was a problem hiding this comment.
thanks @gigiblender! code looks good but since Docker image updates happen only merging the docker/* changes need to be split out into a separate PR that we merge first
| echo set\(CMAKE_FIND_ROOT_PATH_MODE_PROGRAM NEVER\) >> config.cmake | ||
| echo set\(CMAKE_FIND_ROOT_PATH_MODE_LIBRARY ONLY\) >> config.cmake | ||
| else | ||
| # This usually runs in the ci_arm docker image. |
There was a problem hiding this comment.
Should this just error out instead of building since it's not really cross compiling if its run on aarch64
There was a problem hiding this comment.
I am building the cpp tests on aarch64 instead of cross-compiling, and in order to build (and run) the tests, I need the LLVM backend enabled. I am running this config file on the aarch64 Jenkins node before building the tests.
The reason I am not cross-compiling the cpp tests is that I would need to cross-compile gtest beforehand.
Thanks @driazati. I opened #13776 for the docker image update. |
gigiblender
force-pushed
the
cross-build-arm
branch
2 times, most recently
from
a925b5e to
3aef5ac
Compare
This updates ci_minimal to use the changes following on to apache#13714
|
Looks like it causes the |
|
No for some reason Jenkins is ignoring the filter on that job which is supposed to keep it from running on anything but this PR. #13800 should fix it though |
This updates ci_minimal to use the changes following on to apache#13714
This updates ci_minimal to use the changes following on to #13714 Co-authored-by: driazati <[email protected]>
…pache#13776) This PR is a prerequisite to apache#13714 and needs to be merged before. It contains the changes to the ci_minimal docker image to support minimal cross-compilation of TVM to aarch64.
) ~~This is a PR to test the CI~~ This PR is part of apache#13526. It adds cross-compilation of libtvm, libtvm_runtime and libtvm_allvisible to aarch64. It then executes the cpp and python tests on aarch64. A successful run of the CI using the updated minimal image can be found [here](https://ci.tlcpack.ai/blue/organizations/jenkins/tvm-minimal-cross-isa/detail/PR-13714/13/pipeline/60).
This updates ci_minimal to use the changes following on to apache#13714 Co-authored-by: driazati <[email protected]>
This is a PR to test the CIThis PR is part of #13526. It adds cross-compilation of libtvm, libtvm_runtime and libtvm_allvisible to aarch64. It then executes the cpp and python tests on aarch64.
A successful run of the CI using the updated minimal image can be found here.