Skip to content

Commit

Permalink
workflows: test with address sanitizer to catch memory safety issues
Browse files Browse the repository at this point in the history
This enables address sanitizer for CI debug builds on all distributions,
which has proven useful in tracing undefined behaviour that only occurs
on an arbitrary but reproducible subset of distributions.

See #194

Signed-off-by: Peter Colberg <peter.colberg@intel.com>
  • Loading branch information
pcolberg committed Nov 8, 2022
1 parent 900e86e commit 91fcfd3
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion .github/workflows/build-linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,9 @@ jobs:

- name: create build files
run: |
test ${{ matrix.config }} = Debug && ASAN=1
cd build
cmake -G Ninja .. -DCMAKE_BUILD_TYPE=${{ matrix.config }} -DCMAKE_INSTALL_PREFIX=/fpga-runtime-for-opencl -DCMAKE_INSTALL_LIBDIR=lib
cmake -G Ninja .. -DCMAKE_BUILD_TYPE=${{ matrix.config }} -DCMAKE_INSTALL_PREFIX=/fpga-runtime-for-opencl -DCMAKE_INSTALL_LIBDIR=lib ${ASAN:+-DACL_WITH_ASAN=ON}
env:
CC: ${{ matrix.cross }}gcc
CXX: ${{ matrix.cross }}g++
Expand Down

0 comments on commit 91fcfd3

Please # to comment.