Skip to content

Commit

Permalink
fix: force sanitizers to use clang
Browse files Browse the repository at this point in the history
  • Loading branch information
c-dilks committed Feb 14, 2024
1 parent e145b4b commit b80996e
Showing 1 changed file with 3 additions and 10 deletions.
13 changes: 3 additions & 10 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -206,16 +206,9 @@ jobs:
- san_undefined
- san_memory
- san_leak
include:
- { mode: coverage, CC: gcc, CXX: g++ } # TODO: use clang or g++?
- { mode: san_address, CC: gcc, CXX: g++ }
- { mode: san_thread, CC: gcc, CXX: g++ }
- { mode: san_undefined, CC: gcc, CXX: g++ }
- { mode: san_memory, CC: gcc, CXX: g++ }
- { mode: san_leak, CC: gcc, CXX: g++ }
env:
CC: ${{ matrix.CC }}
CXX: ${{ matrix.CXX }}
CC: clang # prefer `clang` sanitizers
CXX: clang++
steps:
### setup
- uses: actions/checkout@v4
Expand Down Expand Up @@ -302,7 +295,7 @@ jobs:
matrix:
binding: [ cpp-gcc, python ]
include:
- { binding: cpp-cpp, extension: '' }
- { binding: cpp-gcc, extension: '' }
- { binding: python, extension: '.py' }
steps:
### dependencies and test data
Expand Down

0 comments on commit b80996e

Please sign in to comment.