Skip to content

Commit

Permalink
CI: Build the full slang-wasm.[js|wasm] targets instead of just the s…
Browse files Browse the repository at this point in the history
…lang library. (#5998)

A recent build breakage for slang-wasm.[js|wasm] was not noticed by CI since it built the
slang library target instead of the slang-wasm.js "executable" target.
(We added a that used objcopy to split debug info from executables. objcopy does not
report errors when it finds object files with unexpected format inside static libraries,
but it *does* report errors when it's run on an exceutable of unexpected format, such as
in the case of slang-wasm.js.)

This closes #5959.
  • Loading branch information
aleino-nv authored Jan 5, 2025
1 parent 00e7542 commit f0431b8
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -125,10 +125,9 @@ jobs:
mkdir generators
cmake --install build --prefix generators --component generators
emcmake cmake -DSLANG_GENERATORS_PATH=generators/bin --preset emscripten -DSLANG_SLANG_LLVM_FLAVOR=DISABLE
cmake --build --preset emscripten --config "$cmake_config" --target slang
[ -f "build.em/$cmake_config/lib/libslang.a" ]
[ -f "build.em/$cmake_config/lib/libcompiler-core.a" ]
[ -f "build.em/$cmake_config/lib/libcore.a" ]
cmake --build --preset emscripten --config "$cmake_config" --target slang-wasm
[ -f "build.em/$cmake_config/bin/slang-wasm.wasm" ]
[ -f "build.em/$cmake_config/bin/slang-wasm.js" ]
else
if [[ "${{ matrix.os }}" =~ "windows" && "${{ matrix.config }}" != "release" && "${{ matrix.config }}" != "releaseWithDebugInfo" ]]; then
# Doing a debug build will try to link against a release built llvm, this
Expand Down

0 comments on commit f0431b8

Please sign in to comment.