[compiler-rt] cd8bba9 - Revert "[compiler-rt] Improve defaults for Android"

Shoaib Meenai via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 29 10:39:09 PDT 2023


Author: Shoaib Meenai
Date: 2023-08-29T10:37:49-07:00
New Revision: cd8bba941f06e17cfdab8cb2b31e9171f019fe29

URL: https://github.com/llvm/llvm-project/commit/cd8bba941f06e17cfdab8cb2b31e9171f019fe29
DIFF: https://github.com/llvm/llvm-project/commit/cd8bba941f06e17cfdab8cb2b31e9171f019fe29.diff

LOG: Revert "[compiler-rt] Improve defaults for Android"

This reverts commit cf403c10fba72ddc6a083a0e1603974d0749c6b3.

This is breaking Android sanitizer buildbots (see the discussion on
https://reviews.llvm.org/D158793).

Added: 
    

Modified: 
    compiler-rt/CMakeLists.txt

Removed: 
    


################################################################################
diff  --git a/compiler-rt/CMakeLists.txt b/compiler-rt/CMakeLists.txt
index d1409318ee0a66..3888995811d894 100644
--- a/compiler-rt/CMakeLists.txt
+++ b/compiler-rt/CMakeLists.txt
@@ -197,7 +197,7 @@ macro(handle_default_cxx_lib var)
     if (APPLE OR CMAKE_SYSTEM_NAME MATCHES "FreeBSD")
       set(${var}_LIBNAME "libc++")
       set(${var}_SYSTEM 1)
-    elseif (ANDROID OR FUCHSIA)
+    elseif (FUCHSIA)
       set(${var}_LIBNAME "libc++")
       set(${var}_INTREE 1)
     else()
@@ -227,19 +227,13 @@ set(CXXLIBS none default libstdc++ libc++)
 set_property(CACHE SANITIZER_TEST_CXX PROPERTY STRINGS ;${CXXLIBS})
 handle_default_cxx_lib(SANITIZER_TEST_CXX)
 
-set(DEFAULT_COMPILER_RT_USE_LLVM_UNWINDER OFF)
-if (ANDROID)
-  set(DEFAULT_COMPILER_RT_USE_LLVM_UNWINDER ON)
-endif()
-
-option(COMPILER_RT_USE_LLVM_UNWINDER
-  "Use the LLVM unwinder." ${DEFAULT_COMPILER_RT_USE_LLVM_UNWINDER})
+option(COMPILER_RT_USE_LLVM_UNWINDER "Use the LLVM unwinder." OFF)
 cmake_dependent_option(COMPILER_RT_ENABLE_STATIC_UNWINDER
   "Statically link the LLVM unwinder." OFF
   "COMPILER_RT_USE_LLVM_UNWINDER" OFF)
 
 set(DEFAULT_SANITIZER_USE_STATIC_LLVM_UNWINDER OFF)
-if (ANDROID OR FUCHSIA)
+if (FUCHSIA)
   set(DEFAULT_SANITIZER_USE_STATIC_LLVM_UNWINDER ON)
 elseif (DEFINED LIBUNWIND_ENABLE_SHARED AND NOT LIBUNWIND_ENABLE_SHARED)
   set(DEFAULT_SANITIZER_USE_STATIC_LLVM_UNWINDER ON)
@@ -277,7 +271,7 @@ cmake_dependent_option(COMPILER_RT_STATIC_CXX_LIBRARY
   "COMPILER_RT_CXX_LIBRARY" OFF)
 
 set(DEFAULT_COMPILER_RT_USE_BUILTINS_LIBRARY OFF)
-if (ANDROID OR FUCHSIA)
+if (FUCHSIA)
   set(DEFAULT_COMPILER_RT_USE_BUILTINS_LIBRARY ON)
 endif()
 


        


More information about the llvm-commits mailing list