Skip to content

Commit 88a681c

Browse files
committed
Use Clang for Hermes builds in CI
- Add explicit Clang compiler flags to Hermes build step - Update cache key to include compiler (prevents mixing GCC/Clang builds) - Ensures consistent compiler usage across Hermes and imgui-react-runtime builds
1 parent 6ea322e commit 88a681c

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

.github/workflows/build.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ jobs:
5353
path: |
5454
hermes-src
5555
hermes-build
56-
key: ${{ runner.os }}-hermes-${{ env.HERMES_VERSION }}
56+
key: ${{ runner.os }}-hermes-${{ env.HERMES_VERSION }}-clang
5757

5858
- name: Build Hermes (if not cached)
5959
if: steps.hermes-cache.outputs.cache-hit != 'true'
@@ -64,7 +64,11 @@ jobs:
6464
# Remove .git directory immediately to save ~100MB in cache
6565
rm -rf .git
6666
cd ..
67-
cmake -S hermes-src -B hermes-build -DCMAKE_BUILD_TYPE=Release -G Ninja
67+
cmake -S hermes-src -B hermes-build \
68+
-DCMAKE_BUILD_TYPE=Release \
69+
-DCMAKE_C_COMPILER=clang \
70+
-DCMAKE_CXX_COMPILER=clang++ \
71+
-G Ninja
6872
cmake --build hermes-build --config Release
6973
7074
- name: Install npm dependencies

0 commit comments

Comments
 (0)