-
Notifications
You must be signed in to change notification settings - Fork 108
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: integrate macOS #519
Comments
A simple CI pipeline for system emulation: ...
+macOS:
+ runs-on: macos-latest
+ steps:
+ - uses: actions/checkout@v4
+ - name: install-dependencies
+ run: |
+ brew install make dtc expect
+ - name: boot Linux kernel test
+ run: |
+ make clean && make ENABLE_SYSTEM=1 && make ENABLE_SYSTEM=1 artifact -j$(sysctl -n hw.logicalcpu)
+ .ci/boot-linux.sh
+ make ENABLE_SYSTEM=1 clean
... |
The occasional failure could be fixed by #536. |
CI breakage on macOS/M1 when running JIT test: CI link It obviously shown that the function is not called ever. @vacantron Can you check it? |
It seems like it's been a long time since the last test on Arm64/MacOS. I'll fix it later. |
I will try to integrate the minimal runnable CI for macOS first. |
Shall we support macOS/Intel? The CI [1] breaks since it expects llvm18 library installed at path Additionally, the behavior of |
Since Apple Silicon (Arm64) has become the predominant hardware platform for macOS, we should focus our CI pipeline on macOS/Arm64 rather than maintaining x64 support. |
diff --git a/mk/common.mk b/mk/common.mk
index 2c721a6..c03dd7e 100644
--- a/mk/common.mk
+++ b/mk/common.mk
@@ -10,6 +10,8 @@ ifeq ($(UNAME_M),x86_64)
HOST_PLATFORM := x86
else ifeq ($(UNAME_M),aarch64)
HOST_PLATFORM := aarch64
+else ifeq ($(UNAME_M),arm64)
+ HOST_PLATFORM := arm64
else
$(error Unsupported platform.)
endif |
Review the |
The current CI pipeline utilizes Linux-based x86-64 and Aarch64 machines for verification and validation (V&V). To enhance the portability of rv32emu, it is both reasonable and feasible to integrate macOS into the CI pipeline.
The text was updated successfully, but these errors were encountered: