[compiler-rt] 6919b70 - [CMake] Set UBSAN_LINK_FLAGS for ubsan

Petr Hosek via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 17 23:22:14 PDT 2020


Author: Petr Hosek
Date: 2020-04-17T23:21:59-07:00
New Revision: 6919b708a180b8791d8d21b289958f1f061c71e3

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

LOG: [CMake] Set UBSAN_LINK_FLAGS for ubsan

This variable is being used, but it's not being set (it's only set
for ubsan_minimal, but not ubsan). This addresses a regression that
was introduced in D78325.

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

Added: 
    

Modified: 
    compiler-rt/lib/ubsan/CMakeLists.txt

Removed: 
    


################################################################################
diff  --git a/compiler-rt/lib/ubsan/CMakeLists.txt b/compiler-rt/lib/ubsan/CMakeLists.txt
index 164f7aadcc0c..174a216c8804 100644
--- a/compiler-rt/lib/ubsan/CMakeLists.txt
+++ b/compiler-rt/lib/ubsan/CMakeLists.txt
@@ -52,6 +52,8 @@ set(UBSAN_CXXFLAGS ${SANITIZER_COMMON_CFLAGS})
 append_rtti_flag(ON UBSAN_CXXFLAGS)
 append_list_if(SANITIZER_CAN_USE_CXXABI -DUBSAN_CAN_USE_CXXABI UBSAN_CXXFLAGS)
 
+set(UBSAN_LINK_FLAGS ${SANITIZER_COMMON_LINK_FLAGS})
+
 set(UBSAN_DYNAMIC_LIBS ${SANITIZER_CXX_ABI_LIBRARIES} ${SANITIZER_COMMON_LINK_LIBS})
 
 append_list_if(COMPILER_RT_HAS_LIBDL dl UBSAN_DYNAMIC_LIBS)


        


More information about the llvm-commits mailing list