Skip to content

Commit

Permalink
Use Ninja for emscripten build of llvm
Browse files Browse the repository at this point in the history
  • Loading branch information
mcbarton authored Jan 10, 2025
1 parent d504612 commit 6a8ad6a
Showing 1 changed file with 6 additions and 8 deletions.
14 changes: 6 additions & 8 deletions .github/workflows/emscripten.yml
Original file line number Diff line number Diff line change
Expand Up @@ -253,14 +253,15 @@ jobs:
brew link --overwrite "$pkg"
done
brew upgrade openssl >/dev/null 2>&1
brew install ninja
brew upgrade
- name: Install deps on Linux
if: ${{ runner.os == 'Linux' && steps.cache.outputs.cache-hit != 'true' }}
run: |
# Install deps
sudo apt-get update
sudo apt-get install valgrind
sudo apt-get install valgrind ninja-build
sudo apt-get autoremove
sudo apt-get clean
Expand Down Expand Up @@ -302,11 +303,9 @@ jobs:
-DLLVM_INCLUDE_EXAMPLES=OFF \
-DLLVM_INCLUDE_TESTS=OFF \
-DLLVM_ENABLE_THREADS=OFF \
-G Ninja \
../llvm
emmake make clang -j ${{ env.ncpus }}
emmake make cling -j ${{ env.ncpus }}
# Now build gtest.a and gtest_main for CppInterOp to run its tests.
emmake make gtest_main -j ${{ env.ncpus }}
emmake ninja clang cling lld gtest_main -j ${{ env.ncpus }}
else
# Apply patches
llvm_vers=$(echo "${{ matrix.clang-runtime }}" | tr '[:lower:]' '[:upper:]')
Expand All @@ -331,10 +330,9 @@ jobs:
-DLLVM_INCLUDE_EXAMPLES=OFF \
-DLLVM_INCLUDE_TESTS=OFF \
-DLLVM_ENABLE_THREADS=OFF \
-G Ninja \
../llvm
emmake make clang -j ${{ env.ncpus }}
emmake make clang-repl -j ${{ env.ncpus }}
emmake make lld -j ${{ env.ncpus }}
emmake ninja clang clang-repl lld -j ${{ env.ncpus }}
fi
cd ../..
Expand Down

0 comments on commit 6a8ad6a

Please sign in to comment.