Skip to content

Commit 8220a79

Browse files
fjtrujyclaude
andcommitted
Update CI/CD to work with ASAN-by-default build
Since ASAN is now enabled by default in CMakeLists.txt, update the CI/CD workflow to: 1. For sanitizer=asan builds: Keep ASAN (enabled by default) and add UBSan 2. For sanitizer=none builds: Explicitly disable ASAN with -DENABLE_ASAN=OFF This ensures we test both configurations: - With ASAN (100% stable, recommended for production) - Without ASAN (for performance comparison and compatibility testing) Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent bebd37e commit 8220a79

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

.github/workflows/compilation.yml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,8 +61,12 @@ jobs:
6161
export PS2DEV=$PWD/ps2dev
6262
export PATH="$PS2DEV/bin:$PATH"
6363
if [ "${{ matrix.sanitizer }}" = "asan" ]; then
64-
export SAN="-fsanitize=address,undefined"
64+
# ASAN is enabled by default in CMakeLists.txt, just add UBSan
65+
export SAN="-fsanitize=undefined"
6566
export EXTRA_CMAKE_FLAGS="-DCMAKE_C_FLAGS='$SAN' -DCMAKE_EXE_LINKER_FLAGS='$SAN'"
67+
else
68+
# Explicitly disable ASAN for non-sanitizer builds to test without it
69+
export EXTRA_CMAKE_FLAGS="-DENABLE_ASAN=OFF"
6670
fi
6771
cmake -B build-cmake -DCMAKE_BUILD_TYPE=${{ matrix.debug }} -DCMAKE_INSTALL_PREFIX=$PS2DEV ${EXTRA_CMAKE_FLAGS}
6872
cmake --build build-cmake --config ${{ matrix.debug }}

0 commit comments

Comments
 (0)