Skip to content

Commit

Permalink
CI: Enable CLMUL in address sanitization test.
Browse files Browse the repository at this point in the history
The crc64_clmul() function should be ignored by the address sanitizer
now so these builds should still pass.
  • Loading branch information
JiaT75 committed Sep 14, 2023
1 parent f167e79 commit 953e775
Showing 1 changed file with 4 additions and 8 deletions.
12 changes: 4 additions & 8 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -78,19 +78,15 @@ jobs:
./build-aux/ci_build.sh -b autotools -p test -f "-m32" -n 32_bit
cd ../xz_build && make distclean
# ifunc and clmul must be disabled for this test because they will
# fail with -fsanitize=address. __attribute__ ifunc is incompatible
# with -fsanitize=address.CLMUL optimizations will read past the
# bounds of small buffers and mask out the unneeded values. This
# triggers -fsanitize=address to report an error even though the
# operation is safe.
# ifunc must be disabled for this test because __attribute__ ifunc is
# incompatible with -fsanitize=address.
- name: Build with -fsanitize=address,undefined
if: ${{ matrix.os == 'ubuntu-latest' && matrix.build_system == 'autotools' }}
run: ./build-aux/ci_build.sh -b autotools -p build -f "-fsanitize=address,undefined" -d ifunc,clmul
run: ./build-aux/ci_build.sh -b autotools -p build -f "-fsanitize=address,undefined" -d ifunc
- name: Test with -fsanitize=address,undefined
if: ${{ matrix.os == 'ubuntu-latest' && matrix.build_system == 'autotools' }}
run: |
./build-aux/ci_build.sh -b autotools -p test -f "-fsanitize=address,undefined" -d ifunc,clmul
./build-aux/ci_build.sh -b autotools -p test -f "-fsanitize=address,undefined" -d ifunc
cd ../xz_build && make distclean
- name: Build with full features
Expand Down

0 comments on commit 953e775

Please sign in to comment.