[compiler-rt] r322999 - [scudo] Pass SANITIZER_COMMON_LINK_FLAGS to the shared library LINK_FLAGS

[Kostya Kortchinsky via llvm-commits]

Author: cryptoad
Date: Fri Jan 19 14:17:39 2018
New Revision: 322999

URL: http://llvm.org/viewvc/llvm-project?rev=322999&view=rev
Log:
[scudo] Pass SANITIZER_COMMON_LINK_FLAGS to the shared library LINK_FLAGS

Summary:
We somehow never did it, and it raised no issue until now, when trying to
enable Fuchsia as a supported Scudo platform in the cmake config.

So propagate SANITIZER_COMMON_LINK_FLAGS for now.

Reviewers: alekseyshl, flowerhack

Reviewed By: flowerhack

Subscribers: mgorny, #sanitizers, llvm-commits

Differential Revision: https://reviews.llvm.org/D42314

Modified:
    compiler-rt/trunk/lib/scudo/CMakeLists.txt

Modified: compiler-rt/trunk/lib/scudo/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/scudo/CMakeLists.txt?rev=322999&r1=322998&r2=322999&view=diff
==============================================================================
--- compiler-rt/trunk/lib/scudo/CMakeLists.txt (original)
+++ compiler-rt/trunk/lib/scudo/CMakeLists.txt Fri Jan 19 14:17:39 2018
@@ -7,6 +7,8 @@ set(SCUDO_CFLAGS ${SANITIZER_COMMON_CFLA
 list(APPEND SCUDO_CFLAGS -fbuiltin)
 append_rtti_flag(OFF SCUDO_CFLAGS)
 
+set(SCUDO_DYNAMIC_LINK_FLAGS ${SANITIZER_COMMON_LINK_FLAGS})
+
 set(SCUDO_SOURCES
   scudo_allocator.cpp
   scudo_flags.cpp
@@ -68,6 +70,7 @@ if(COMPILER_RT_HAS_SCUDO)
                 RTUbsan
                 RTUbsan_cxx
     CFLAGS ${SCUDO_CFLAGS}
+    LINK_FLAGS ${SCUDO_DYNAMIC_LINK_FLAGS}
     LINK_LIBS ${SCUDO_DYNAMIC_LIBS}
     PARENT_TARGET scudo)
 endif()