Revert r282483 - [cmake] Add linker option "-Wl,-z,defs" in standalone build
authorMichal Gorny <mgorny@gentoo.org>
Tue, 27 Sep 2016 18:54:02 +0000 (18:54 +0000)
committerMichal Gorny <mgorny@gentoo.org>
Tue, 27 Sep 2016 18:54:02 +0000 (18:54 +0000)
Revert r282483 as it causes build failures due to missing symbols when
not linking to -lgcc_s (i.e. doing pure LLVM stack build). The patch can
be reintroduced when the build system is fixed to add all needed
libraries (libunwind, compiler-rt).

llvm-svn: 282524

libcxx/CMakeLists.txt

index bc4aa30..2cf5ced 100644 (file)
@@ -319,18 +319,6 @@ remove_flags(-stdlib=libc++ -stdlib=libstdc++)
 # so they don't get transformed into -Wno and -errors respectivly.
 remove_flags(-Wno-pedantic -pedantic-errors -pedantic)
 
-# FIXME: this is cribbed from HandleLLVMOptions.cmake.
-if(LIBCXX_STANDALONE_BUILD)
-  # Pass -Wl,-z,defs. This makes sure all symbols are defined. Otherwise a DSO
-  # build might work on ELF but fail on MachO/COFF.
-  if(NOT (${CMAKE_SYSTEM_NAME} MATCHES "Darwin" OR WIN32 OR CYGWIN OR
-          ${CMAKE_SYSTEM_NAME} MATCHES "FreeBSD" OR
-          ${CMAKE_SYSTEM_NAME} MATCHES "OpenBSD") AND
-     NOT LLVM_USE_SANITIZER)
-    set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} -Wl,-z,defs")
-  endif()
-endif()
-
 # Required flags ==============================================================
 set(LIBCXX_STANDARD_VER c++11 CACHE INTERNAL "internal option to change build dialect")
 add_compile_flags_if_supported(-std=${LIBCXX_STANDARD_VER})