[arrow] branch master updated: ARROW-12130: [C++] Don't enable Neon if -DARROW_SIMD_LEVEL=NONE

[kou]

This is an automated email from the ASF dual-hosted git repository.

kou pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new 3cfd611  ARROW-12130: [C++] Don't enable Neon if 
-DARROW_SIMD_LEVEL=NONE
3cfd611 is described below

commit 3cfd6112843ba6d5a3c16e4ad83d788925ebd508
Author: Yibo Cai <yibo....@arm.com>
AuthorDate: Mon Mar 29 14:11:51 2021 +0900

    ARROW-12130: [C++] Don't enable Neon if -DARROW_SIMD_LEVEL=NONE
    
    Closes #9830 from cyb70289/neon-simd-none
    
    Authored-by: Yibo Cai <yibo....@arm.com>
    Signed-off-by: Sutou Kouhei <k...@clear-code.com>
---
 cpp/cmake_modules/SetupCxxFlags.cmake | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/cpp/cmake_modules/SetupCxxFlags.cmake 
b/cpp/cmake_modules/SetupCxxFlags.cmake
index b534552..9f68c56 100644
--- a/cpp/cmake_modules/SetupCxxFlags.cmake
+++ b/cpp/cmake_modules/SetupCxxFlags.cmake
@@ -451,7 +451,9 @@ if(ARROW_CPU_FLAG STREQUAL "armv8")
   endif()
   set(CXX_COMMON_FLAGS "${CXX_COMMON_FLAGS} ${ARROW_ARMV8_ARCH_FLAG}")
 
-  add_definitions(-DARROW_HAVE_NEON)
+  if(NOT ARROW_SIMD_LEVEL STREQUAL "NONE")
+    add_definitions(-DARROW_HAVE_NEON)
+  endif()
 
   if(CMAKE_CXX_COMPILER_ID STREQUAL "GNU"
      AND CMAKE_CXX_COMPILER_VERSION VERSION_LESS "5.4")