Skip to content
This repository has been archived by the owner on Apr 5, 2019. It is now read-only.

Commit

Permalink
.travis.yml: Disable ASAN for Clang/Linux
Browse files Browse the repository at this point in the history
Still trying to track down the pesky sporadic segfault in
`ristretto_gen_tables`
  • Loading branch information
tarcieri committed Aug 15, 2018
1 parent fd7ef8d commit ea998d6
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 19 deletions.
20 changes: 2 additions & 18 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,10 @@
language: c
sudo: required

install:
- curl https://sh.rustup.rs -sSf | sh -s -- -y
- export PATH=$PATH:~/.cargo/bin
- sudo apt-get install lldb-3.3

env:
- XCFLAGS=-g
Expand All @@ -19,24 +21,6 @@ os:
- osx
- linux

matrix:
fast_finish: true
exclude:
- os: linux
compiler: clang
include:
- os: linux
compiler: clang
script:
# Enable ASAN as a workaround for ristretto_gen_tables SEGV
# See: https://github.com/Ristretto/libristretto255/issues/24
- XCFLAGS="-g -fsanitize=address" XLDFLAGS=-fsanitize=address ASAN_OPTIONS=detect_leaks=0 make
- make clean
- make build/obj/bin/ristretto_gen_tables && rm build/obj/ristretto_gen_tables.o
- clang -std=c99 -fno-strict-aliasing -g -Iinclude -Isrc -Isrc/arch/x86_64 -O2 -march=native -ffunction-sections -fdata-sections -fomit-frame-pointer -fPIC -c -o build/obj/ristretto_tables.o src/ristretto_tables.c
- ar rcs build/lib/libristretto255.a build/obj/*.o
- cd tests && cargo test --all --lib

branches:
only:
- master
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ $(BUILD_IBIN)/ristretto_gen_tables: $(GENCOMPONENTS)
$(LD) $(LDFLAGS) -o $@ $^

src/ristretto_tables.c: $(BUILD_IBIN)/ristretto_gen_tables
./$< > $@ || (rm $@; exit 1)
echo "run" | lldb-3.3 ./$< > $@ || (rm $@; exit 1)

# The libristretto255 library
lib: $(BUILD_LIB)/libristretto255.so $(BUILD_LIB)/libristretto255.a
Expand Down

0 comments on commit ea998d6

Please sign in to comment.