You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2023/01/13 19:05:12 UTC

[kudu] branch master updated: Skips -msse4.2 compiler flag for arch: ppc64le

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 5cdcb709e Skips -msse4.2 compiler flag for arch: ppc64le
5cdcb709e is described below

commit 5cdcb709e3d180518aba22724ec3ad0b8e38dcd9
Author: Hari Reddy <hn...@us.ibm.com>
AuthorDate: Fri Sep 16 12:44:57 2022 -0500

    Skips -msse4.2 compiler flag for arch: ppc64le
    
    Files affected kudu/CMakeLists.txt
    
    sse4.2 is x86_64 specific compiler flag. Since compilers for powerpc
    architecture (ppc64le) do not support this flag, this patch skips
     this flag when  ARCH_NAME=ppc64le is detected.
    
     Changes to be committed:
            modified:   CMakeLists.txt
    
    Change-Id: Ic5e87857796e97fffc85c527045b7e4fa914e896
    Reviewed-on: http://gerrit.cloudera.org:8080/19000
    Tested-by: Kudu Jenkins
    Reviewed-by: Alexey Serbin <al...@apache.org>
---
 CMakeLists.txt | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 70e6bcd9d..91bd2fe6c 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -189,7 +189,7 @@ if("${ARCH_NAME}" MATCHES "aarch64")
   set(CXX_COMMON_FLAGS "${CXX_COMMON_FLAGS} -fsigned-char")
   # Turn off fp-contract on aarch64 to avoid multiply-add operation result difference.
   set(CXX_COMMON_FLAGS "${CXX_COMMON_FLAGS} -ffp-contract=off")
-else()
+elseif( NOT ( "${ARCH_NAME}" MATCHES "ppc64le" ))
   # -msse4.2: Enable sse4.2 compiler intrinsics.
   set(CXX_COMMON_FLAGS "-msse4.2")
 endif()