Skip to content

Commit da87956

Browse files
authored
Merge pull request #54 from jcarpent/topic/cmake
Don't compile AVX Python bindings on non X86_64 arch
2 parents 648a27f + f67cc13 commit da87956

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ set(CMAKE_MODULE_PATH
6767
set(CMAKE_MODULE_PATH "${CMAKE_CURRENT_LIST_DIR}/cmake-external"
6868
${CMAKE_MODULE_PATH})
6969

70+
message("-- CMAKE_SYSTEM_PROCESSOR: ${CMAKE_SYSTEM_PROCESSOR}")
7071
if(INITIALIZE_EIGEN_WITH_NAN)
7172
add_definitions(-DEIGEN_INITIALIZE_MATRICES_BY_NAN)
7273
endif(INITIALIZE_EIGEN_WITH_NAN)

bindings/python/CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ file(GLOB_RECURSE PYWRAP_HEADERS ${CMAKE_CURRENT_LIST_DIR}/src/*.hpp)
1919
file(GLOB_RECURSE PYWRAP_SOURCES ${CMAKE_CURRENT_LIST_DIR}/src/*.cpp)
2020

2121
# Add simd feature detectors for current intel CPU
22-
message("-- CMAKE_SYSTEM_PROCESSOR: ${CMAKE_SYSTEM_PROCESSOR}")
2322
if(${CMAKE_SYSTEM_PROCESSOR} MATCHES "(x86)|(X86)|(amd64)|(AMD64)")
2423
python3_add_library(instructionset MODULE helpers/instruction-set.cpp)
2524
add_dependencies(python instructionset)
@@ -63,7 +62,8 @@ function(CREATE_PYTHON_TARGET target_name COMPILE_OPTIONS dependencies)
6362
endfunction()
6463

6564
create_python_target(proxsuite_pywrap "" proxsuite)
66-
if(BUILD_WITH_VECTORIZATION_SUPPORT)
65+
if(BUILD_WITH_VECTORIZATION_SUPPORT AND ${CMAKE_SYSTEM_PROCESSOR} MATCHES
66+
"(x86)|(X86)|(amd64)|(AMD64)")
6767
if(BUILD_BINDINGS_WITH_AVX2_SUPPORT)
6868
create_python_target(proxsuite_pywrap_avx2 "-mavx2;-mfma"
6969
proxsuite-vectorized)

0 commit comments

Comments
 (0)