Skip to content

Commit

Permalink
CI, SIMD: Test and build without the support of AVX2 and AVX512
Browse files Browse the repository at this point in the history
  Most CI nodes have the support of AVX 512, and AVX2 which
  leaves the SSE SIMD kernels untested and only counts on
  internal universal intrinsic tests wich may not be enough.
  • Loading branch information
seiko2plus committed Sep 22, 2022
1 parent 54f9b4a commit 73e887a
Show file tree
Hide file tree
Showing 2 changed files with 32 additions and 0 deletions.
30 changes: 30 additions & 0 deletions .github/workflows/build_test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,36 @@ jobs:
python-version: ${{ env.PYTHON_VERSION }}
- uses: ./.github/actions

without_avx512:
needs: [smoke_test]
runs-on: ubuntu-latest
env:
CPU_DISPATCH: "max -xop -fma4 -avx512f -avx512cd -avx512_knl -avx512_knm -avx512_skx -avx512_clx -avx512_cnl -avx512_icl"
steps:
- uses: actions/checkout@v3
with:
submodules: recursive
fetch-depth: 0
- uses: actions/setup-python@v3
with:
python-version: ${{ env.PYTHON_VERSION }}
- uses: ./.github/actions

without_avx512_avx2_fma3:
needs: [smoke_test]
runs-on: ubuntu-latest
env:
CPU_DISPATCH: "SSSE3 SSE41 POPCNT SSE42 AVX F16C"
steps:
- uses: actions/checkout@v3
with:
submodules: recursive
fetch-depth: 0
- uses: actions/setup-python@v3
with:
python-version: ${{ env.PYTHON_VERSION }}
- uses: ./.github/actions

debug:
needs: [smoke_test]
runs-on: ubuntu-20.04
Expand Down
2 changes: 2 additions & 0 deletions tools/travis-test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,8 @@ setup_base()
# only disable SIMD optimizations
elif [ -n "$WITHOUT_SIMD" ]; then
build_args+=("--cpu-baseline=none" "--cpu-dispatch=none")
elif [ -n "$CPU_DISPATCH" ]; then
build_args+=("--cpu-dispatch=$CPU_DISPATCH")
else
# SIMD extensions that need to be tested on both runtime and compile-time via (test_simd.py)
# any specified features will be ignored if they're not supported by compiler or platform
Expand Down

0 comments on commit 73e887a

Please sign in to comment.