[scudo] Pass SANITIZER_COMMON_LINK_FLAGS to the shared library LINK_FLAGS
authorKostya Kortchinsky <kostyak@google.com>
Fri, 19 Jan 2018 22:17:39 +0000 (22:17 +0000)
committerKostya Kortchinsky <kostyak@google.com>
Fri, 19 Jan 2018 22:17:39 +0000 (22:17 +0000)
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

llvm-svn: 322999

compiler-rt/lib/scudo/CMakeLists.txt

index f86de6a..b612fc5 100644 (file)
@@ -7,6 +7,8 @@ set(SCUDO_CFLAGS ${SANITIZER_COMMON_CFLAGS})
 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()