Skip to content

Commit

Permalink
Disable AVX512 for aws-lc on old GCC (#468)
Browse files Browse the repository at this point in the history
  • Loading branch information
TingDaoK authored Apr 19, 2023
1 parent 090edb8 commit 80a1f21
Show file tree
Hide file tree
Showing 3 changed files with 27 additions and 2 deletions.
18 changes: 18 additions & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,24 @@ env:
AWS_REGION: us-east-1

jobs:
manylinux1:
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
image:
- x64
- x86
python:
- cp37-cp37m
- cp38-cp38
- cp39-cp39
steps:
- name: Build ${{ env.PACKAGE_NAME }}
run: |
aws s3 cp s3://aws-crt-test-stuff/ci/${{ env.BUILDER_VERSION }}/linux-container-ci.sh ./linux-container-ci.sh && chmod a+x ./linux-container-ci.sh
./linux-container-ci.sh ${{ env.BUILDER_VERSION }} aws-crt-manylinux1-${{ matrix.image }} build -p ${{ env.PACKAGE_NAME }} --python /opt/python/${{ matrix.python }}/bin/python
manylinux2014:
runs-on: ubuntu-20.04 # latest
strategy:
Expand Down
9 changes: 8 additions & 1 deletion crt/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,15 @@ if(UNIX AND NOT APPLE)

if(NOT USE_OPENSSL)
set(DISABLE_GO ON CACHE BOOL "Build without using Go, we don't want the extra dependency")
set(DISABLE_PERL ON CACHE BOOL "Build without using Perl, we don't want the extra dependency")
set(BUILD_LIBSSL OFF CACHE BOOL "Don't need libssl, only need libcrypto")

if(CMAKE_C_COMPILER_ID MATCHES "GNU" AND CMAKE_C_COMPILER_VERSION VERSION_LESS "5.0")
set(DISABLE_PERL OFF CACHE BOOL "Build with Perl to avoid using pre-compiled binary with AVX512")
set(MY_ASSEMBLER_IS_TOO_OLD_FOR_512AVX ON CACHE BOOL "Disable AVX512 on old GCC that not supports it")
else()
set(DISABLE_PERL ON CACHE BOOL "Build without using Perl, we don't want the extra dependency")
endif()

add_subdirectory(aws-lc)
endif()

Expand Down
2 changes: 1 addition & 1 deletion crt/aws-lc

0 comments on commit 80a1f21

Please sign in to comment.