[llvm] r226613 - Don't pass -Wl,z,defs for now.

Rafael Espindola rafael.espindola at gmail.com
Tue Jan 20 14:08:21 PST 2015


Author: rafael
Date: Tue Jan 20 16:08:20 2015
New Revision: 226613

URL: http://llvm.org/viewvc/llvm-project?rev=226613&view=rev
Log:
Don't pass -Wl,z,defs for now.

It broke the msan build.

Modified:
    llvm/trunk/cmake/modules/HandleLLVMOptions.cmake

Modified: llvm/trunk/cmake/modules/HandleLLVMOptions.cmake
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/HandleLLVMOptions.cmake?rev=226613&r1=226612&r2=226613&view=diff
==============================================================================
--- llvm/trunk/cmake/modules/HandleLLVMOptions.cmake (original)
+++ llvm/trunk/cmake/modules/HandleLLVMOptions.cmake Tue Jan 20 16:08:20 2015
@@ -104,13 +104,6 @@ if(APPLE)
   set(CMAKE_MODULE_LINKER_FLAGS "${CMAKE_MODULE_LINKER_FLAGS} -Wl,-flat_namespace -Wl,-undefined -Wl,suppress")
 endif()
 
-# 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))
-  set(CMAKE_SHARED_LINKER_FLAGS "${CMAKE_SHARED_LINKER_FLAGS} -Wl,-z,defs")
-endif()
-
-
 function(append value)
   foreach(variable ${ARGN})
     set(${variable} "${${variable}} ${value}" PARENT_SCOPE)





More information about the llvm-commits mailing list