diff options
| author | Ellie Hermaszewska <ellieh@nvidia.com> | 2024-10-30 10:15:14 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-10-29 19:15:14 -0700 |
| commit | 44dc5ea202ac43cd4f18c268e95143c2a23f5d26 (patch) | |
| tree | 7faeaf99ddda7f5ee223c994f730cac0e8822fe3 /.github/workflows/ci.yml | |
| parent | 613a29affe272772dfe0c463d866fb0b8c1d42ee (diff) | |
Put debug symbols in releases (#5370)
Diffstat (limited to '.github/workflows/ci.yml')
| -rw-r--r-- | .github/workflows/ci.yml | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 3c32db8c6..6f0aa2d93 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -23,7 +23,7 @@ jobs: strategy: matrix: os: [linux, macos, windows] - config: [debug, release] + config: [debug, releaseWithDebugInfo] compiler: [gcc, clang, cl] platform: [x86_64, aarch64, wasm] exclude: @@ -54,14 +54,14 @@ jobs: # quick or full conditionally otherwise - test-category: smoke - { os: windows, test-category: quick } - - { config: release, test-category: full } + - { config: releaseWithDebugInfo, test-category: full } # default not full gpu tests - full-gpu-tests: false # The runners don't have a GPU by default except for the self-hosted ones - has-gpu: false # Self-hosted aarch64 build - os: linux - config: release + config: releaseWithDebugInfo compiler: gcc platform: aarch64 test-category: smoke @@ -70,7 +70,7 @@ jobs: has-gpu: true # Self-hosted full gpu build - os: windows - config: release + config: releaseWithDebugInfo compiler: cl platform: x86_64 test-category: full @@ -109,13 +109,13 @@ jobs: cmake --workflow --preset generators --fresh mkdir generators cmake --install build --prefix generators --component generators - emcmake cmake -DSLANG_GENERATORS_PATH=generators/bin --preset emscripten -G "Ninja" -DSLANG_SLANG_LLVM_FLAVOR=DISABLE - cmake --build --preset emscripten --target slang - [ -f "build.em/Release/lib/libslang.a" ] - [ -f "build.em/Release/lib/libcompiler-core.a" ] - [ -f "build.em/Release/lib/libcore.a" ] + 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" ] else - if [[ "${{ matrix.os }}" =~ "windows" && "${{ matrix.config }}" != "release" ]]; then + 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 # is a problem on Windows, so make slang-llvm in release build and use # that as though it's a fetched binary via these presets. @@ -127,7 +127,7 @@ jobs: "-DCMAKE_COMPILE_WARNING_AS_ERROR=${{matrix.warnings-as-errors}}" cmake --workflow --preset "${{matrix.config}}" else - # Otherwise, use the system llvm we have just build or got from the + # Otherwise, use the "system" llvm we have just build or got from the # cache in the setup phase cmake --preset default --fresh \ -DSLANG_SLANG_LLVM_FLAVOR=USE_SYSTEM_LLVM \ |