[CI] Add m6g instance (ARM64) to CI #6781
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
leandron
reviewed
Oct 28, 2020
Comment on lines
+30
to
+31
| COPY install/ubuntu_install_llvm.sh /install/ubuntu_install_llvm.sh | ||
| RUN bash /install/ubuntu_install_llvm.sh |
There was a problem hiding this comment.
This script uses apt.llvm.org repository, which does not publish AArch64 (arm64) builds. So, if you want to use the same LLVM versions we have for other builds, for now, one option would be downloading and uncompressing an LLVM build from the official source:
- LLVM 11: https://github.com/llvm/llvm-project/releases/download/llvmorg-11.0.0/clang+llvm-11.0.0-aarch64-linux-gnu.tar.xz
- LLVM 10: https://github.com/llvm/llvm-project/releases/download/llvmorg-10.0.0/clang+llvm-10.0.0-aarch64-linux-gnu.tar.xz
- LLVM 9: https://releases.llvm.org/9.0.0/clang+llvm-9.0.0-aarch64-linux-gnu.tar.xz
Once arm64 builds are published into apt.llvm.org, then we can just share the same existing scripts.
There was a problem hiding this comment.
The current one is more like experimental. I did test it locally. It seems every was okay. I will probably leave these arm specific installation (including acl etc) to your folks, how do you think?
docker/Dockerfile.ci_arm
Outdated
Comment on lines
23
to
25
| RUN apt-get update --fix-missing | ||
| RUN apt-get install -y ca-certificates | ||
| RUN apt-get install -y gnupg2 |
There was a problem hiding this comment.
This is likely to increase the space taken by the resultant Docker image. It is recommended (by Docker) to line-up apt-get calls in one statement, and make sure you remove the apt cache by the end.
zhiics
added a commit
to zhiics/tvm
that referenced
this pull request
Oct 30, 2020
* [CI] Add m6g instance (ARM64) to CI * address comments Co-authored-by: Ubuntu <ubuntu@ip-172-31-54-90.us-west-2.compute.internal>
tqchen
added a commit
that referenced
this pull request
Oct 31, 2020
* [CI] Add m6g instance (ARM64) to CI (#6781) * [CI] Add m6g instance (ARM64) to CI * address comments Co-authored-by: Ubuntu <ubuntu@ip-172-31-54-90.us-west-2.compute.internal> * [CI] fix cpp test (#6796) * Update tests/python/unittest/test_target_codegen_x86.py Co-authored-by: Cody Yu <comaniac0422@gmail.com> Co-authored-by: Ubuntu <ubuntu@ip-172-31-54-90.us-west-2.compute.internal> Co-authored-by: Tianqi Chen <tqchen@users.noreply.github.com> Co-authored-by: Cody Yu <comaniac0422@gmail.com>
trevor-m
pushed a commit
to trevor-m/tvm
that referenced
this pull request
Dec 2, 2020
* [CI] Add m6g instance (ARM64) to CI (apache#6781) * [CI] Add m6g instance (ARM64) to CI * address comments Co-authored-by: Ubuntu <ubuntu@ip-172-31-54-90.us-west-2.compute.internal> * [CI] fix cpp test (apache#6796) * Update tests/python/unittest/test_target_codegen_x86.py Co-authored-by: Cody Yu <comaniac0422@gmail.com> Co-authored-by: Ubuntu <ubuntu@ip-172-31-54-90.us-west-2.compute.internal> Co-authored-by: Tianqi Chen <tqchen@users.noreply.github.com> Co-authored-by: Cody Yu <comaniac0422@gmail.com>
trevor-m
pushed a commit
to trevor-m/tvm
that referenced
this pull request
Dec 4, 2020
* [CI] Add m6g instance (ARM64) to CI (apache#6781) * [CI] Add m6g instance (ARM64) to CI * address comments Co-authored-by: Ubuntu <ubuntu@ip-172-31-54-90.us-west-2.compute.internal> * [CI] fix cpp test (apache#6796) * Update tests/python/unittest/test_target_codegen_x86.py Co-authored-by: Cody Yu <comaniac0422@gmail.com> Co-authored-by: Ubuntu <ubuntu@ip-172-31-54-90.us-west-2.compute.internal> Co-authored-by: Tianqi Chen <tqchen@users.noreply.github.com> Co-authored-by: Cody Yu <comaniac0422@gmail.com>
trevor-m
pushed a commit
to neo-ai/tvm
that referenced
this pull request
Dec 4, 2020
* [CI] Add m6g instance (ARM64) to CI (apache#6781) * [CI] Add m6g instance (ARM64) to CI * address comments Co-authored-by: Ubuntu <ubuntu@ip-172-31-54-90.us-west-2.compute.internal> * [CI] fix cpp test (apache#6796) * Update tests/python/unittest/test_target_codegen_x86.py Co-authored-by: Cody Yu <comaniac0422@gmail.com> Co-authored-by: Ubuntu <ubuntu@ip-172-31-54-90.us-west-2.compute.internal> Co-authored-by: Tianqi Chen <tqchen@users.noreply.github.com> Co-authored-by: Cody Yu <comaniac0422@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.