From 2e76bd7d1013369f0a9a10d5c01c53ec8f65121f Mon Sep 17 00:00:00 2001 From: Yann Collet Date: Wed, 5 May 2021 20:43:04 -0700 Subject: [PATCH 1/3] attempt to make Appveyor's Cygwin test faster Cygwin is the longest Appveyor test Appveyor is typically the CI which finish last --- appveyor.yml | 4 ++-- build/cmake/tests/CMakeLists.txt | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/appveyor.yml b/appveyor.yml index 172159315..80d5e65b5 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -252,8 +252,8 @@ C:\cygwin64\bin\bash --login -c " set -e; cd build/cmake; - CFLAGS='-Werror' cmake -G 'Unix Makefiles' -DCMAKE_BUILD_TYPE=Debug -DZSTD_BUILD_TESTS:BOOL=ON -DZSTD_FUZZER_FLAGS=-T30s -DZSTD_ZSTREAM_FLAGS=-T30s .; - make -j4; + CFLAGS='-O1 -Werror' cmake -G 'Unix Makefiles' -DCMAKE_BUILD_TYPE=Debug -DZSTD_BUILD_TESTS:BOOL=ON -DZSTD_FUZZER_FLAGS=-T20s -DZSTD_ZSTREAM_FLAGS=-T20s -DZSTD_FULLBENCH_FLAGS=-i0 .; + make VERBOSE=1 -j4; ctest -V -L Medium; " ) diff --git a/build/cmake/tests/CMakeLists.txt b/build/cmake/tests/CMakeLists.txt index 34eca9109..fe095eb6a 100644 --- a/build/cmake/tests/CMakeLists.txt +++ b/build/cmake/tests/CMakeLists.txt @@ -58,7 +58,7 @@ target_link_libraries(datagen libzstd_static) # add_executable(fullbench ${PROGRAMS_DIR}/datagen.c ${PROGRAMS_DIR}/util.c ${PROGRAMS_DIR}/timefn.c ${PROGRAMS_DIR}/benchfn.c ${PROGRAMS_DIR}/benchzstd.c ${TESTS_DIR}/fullbench.c) target_link_libraries(fullbench libzstd_static) -add_test(NAME fullbench COMMAND fullbench) +add_test(NAME fullbench COMMAND fullbench ${ZSTD_FULLBENCH_FLAGS}) # # fuzzer From bd547232bc0aa43591b9eabf51a78ae3cb10fdcf Mon Sep 17 00:00:00 2001 From: Yann Collet Date: Thu, 6 May 2021 16:07:44 -0700 Subject: [PATCH 2/3] switch to clang --- appveyor.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/appveyor.yml b/appveyor.yml index 551a7b44e..877f841f1 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -279,8 +279,8 @@ C:\cygwin64\bin\bash --login -c " set -e; cd build/cmake; - CFLAGS='-O1 -Werror' cmake -G 'Unix Makefiles' -DCMAKE_BUILD_TYPE=Debug -DZSTD_BUILD_TESTS:BOOL=ON -DZSTD_FUZZER_FLAGS=-T20s -DZSTD_ZSTREAM_FLAGS=-T20s -DZSTD_FULLBENCH_FLAGS=-i0 .; - make VERBOSE=1 -j4; + CC=clang CFLAGS='-O1 -Werror' cmake -G 'Unix Makefiles' -DCMAKE_BUILD_TYPE=Debug -DZSTD_BUILD_TESTS:BOOL=ON -DZSTD_FUZZER_FLAGS=-T20s -DZSTD_ZSTREAM_FLAGS=-T20s -DZSTD_FULLBENCH_FLAGS=-i0 .; + make VERBOSE=1 -j; ctest -V -L Medium; " ) From 17b9e43c7d385c165f86fba1053a68b950032d22 Mon Sep 17 00:00:00 2001 From: Yann Collet Date: Thu, 6 May 2021 21:53:30 -0700 Subject: [PATCH 3/3] do not install g++ --- appveyor.yml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/appveyor.yml b/appveyor.yml index 877f841f1..c6ab78688 100644 --- a/appveyor.yml +++ b/appveyor.yml @@ -256,7 +256,6 @@ - if [%HOST%]==[cygwin] ( ECHO Installing Cygwin Packages && C:\cygwin64\setup-x86_64.exe -qnNdO -R "C:\cygwin64" -g -P ^ - gcc-g++,^ gcc,^ cmake,^ make @@ -279,7 +278,7 @@ C:\cygwin64\bin\bash --login -c " set -e; cd build/cmake; - CC=clang CFLAGS='-O1 -Werror' cmake -G 'Unix Makefiles' -DCMAKE_BUILD_TYPE=Debug -DZSTD_BUILD_TESTS:BOOL=ON -DZSTD_FUZZER_FLAGS=-T20s -DZSTD_ZSTREAM_FLAGS=-T20s -DZSTD_FULLBENCH_FLAGS=-i0 .; + CFLAGS='-Werror' cmake -G 'Unix Makefiles' -DCMAKE_BUILD_TYPE=Debug -DZSTD_BUILD_TESTS:BOOL=ON -DZSTD_FUZZER_FLAGS=-T20s -DZSTD_ZSTREAM_FLAGS=-T20s -DZSTD_FULLBENCH_FLAGS=-i0 .; make VERBOSE=1 -j; ctest -V -L Medium; "