From 56c37d7035e34e3da6adc21567b8abf4c73a89ba Mon Sep 17 00:00:00 2001 From: Matan Breizman Date: Sun, 2 Mar 2025 08:42:45 +0000 Subject: [PATCH] cmake/modules/BuildISAL.cmake: set no-integrated-as on clang only this option is only relevant to clang, gcc will fail with: ``` CMake Error at ceph/build/src/erasure-code/isa/isal_ext-prefix/src/isal_ext-stamp/isal_ext-configure-Debug-impl.cmake:19 (message): Command failed (77): 'env' 'CC=/usr/bin/gcc-11' './configure' '--prefix=ceph/build/src/isa-l/install' '--with-pic' '--enable-static' '--disable-shared' 'CFLAGS=-no-integrated-as' ``` Signed-off-by: Matan Breizman --- cmake/modules/BuildISAL.cmake | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmake/modules/BuildISAL.cmake b/cmake/modules/BuildISAL.cmake index 16e2030de1a..fa1476cabf8 100644 --- a/cmake/modules/BuildISAL.cmake +++ b/cmake/modules/BuildISAL.cmake @@ -17,7 +17,7 @@ function(build_isal) # because it messes with the internal install paths of arrow's bundled deps set(NO_DESTDIR_COMMAND ${CMAKE_COMMAND} -E env --unset=DESTDIR) - if(HAVE_ARMV8_SIMD) + if(CMAKE_C_COMPILER_ID MATCHES "Clang" AND HAVE_ARMV8_SIMD) list(APPEND configure_cmd CFLAGS=-no-integrated-as) endif() -- 2.39.5