Skip to content

Commit

Permalink
ci: Combine the asan and ubsan jobs
Browse files Browse the repository at this point in the history
Running these two sanitizers together is fine, so let's do that to save
ourselves a job. This means we're also updating the asan job to use LLVM
15 instead of 14, but that's fine since we still use LLVM 14 for our
tsan tests.
  • Loading branch information
robinlinden committed Dec 2, 2023
1 parent c0cb84f commit 32276bb
Showing 1 changed file with 2 additions and 8 deletions.
10 changes: 2 additions & 8 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,18 +32,12 @@ jobs:
version: 14
bazel: --config tsan

- name: clang-asan
os: ubuntu-22.04
compiler: clang
version: 14
bazel: --config asan

# https://github.com/llvm/llvm-project/issues/49689
- name: clang-ubsan
- name: clang-15-asan-ubsan
os: ubuntu-22.04
compiler: clang
version: 15
bazel: --config ubsan
bazel: --config asan --config ubsan

- name: clang-17
os: ubuntu-22.04
Expand Down

0 comments on commit 32276bb

Please sign in to comment.