Skip to content

Commit

Permalink
ci: verify ASAN presence/absence as appropriate
Browse files Browse the repository at this point in the history
Presently only done on Linux since the test relies on `nm` being
available.
  • Loading branch information
cpu committed Apr 19, 2024
1 parent a097fa1 commit 83e96bf
Showing 1 changed file with 14 additions and 0 deletions.
14 changes: 14 additions & 0 deletions .github/workflows/test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,20 @@ jobs:
run: make CC=${{ matrix.cc }} PROFILE=debug test integration
- name: Platform verifier connect test
run: make PROFILE=debug connect-test
- name: Verify debug builds were using ASAN
if: runner.os == 'Linux' # For 'nm'
run: >
nm target/client | grep '__asan_init'
nm target/server | grep '__asan_init'
- name: Build release binaries
run: >
make clean
make CC=${{ matrix.cc }} PROFILE=release
- name: Verify release builds were not using ASAN
if: runner.os == 'Linux' # For 'nm'
run: >
nm target/client | grep -v '__asan_init'
nm target/server | grep -v '__asan_init'
valgrind:
name: Valgrind
Expand Down

0 comments on commit 83e96bf

Please sign in to comment.