Skip to content
This repository was archived by the owner on May 21, 2019. It is now read-only.

Commit 99ae2fa

Browse files
committed
Revert "[sanitizer] Support compiler-rt builtins"
This reverts commit fd63314. git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@309083 91177308-0d34-0410-b5e6-96231b3b80d8
1 parent 8e31fff commit 99ae2fa

File tree

9 files changed

+7
-82
lines changed

9 files changed

+7
-82
lines changed

CMakeLists.txt

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,6 @@ option(COMPILER_RT_EXTERNALIZE_DEBUGINFO
7878
# COMPILER_RT_DEBUG_PYBOOL is used by lit.common.configured.in.
7979
pythonize_bool(COMPILER_RT_DEBUG)
8080

81-
include(HandleCompilerRT)
8281
include(config-ix)
8382

8483
if(APPLE AND SANITIZER_MIN_OSX_VERSION AND SANITIZER_MIN_OSX_VERSION VERSION_LESS "10.9")
@@ -94,8 +93,6 @@ endif()
9493
option(SANITIZER_CAN_USE_CXXABI "Sanitizers can use cxxabi" ${use_cxxabi_default})
9594
pythonize_bool(SANITIZER_CAN_USE_CXXABI)
9695

97-
option(SANITIZER_USE_COMPILER_RT "Use compiler-rt builtins instead of libgcc" OFF)
98-
9996
#================================
10097
# Setup Compiler Flags
10198
#================================
@@ -231,19 +228,6 @@ append_list_if(COMPILER_RT_HAS_WD4391_FLAG /wd4391 SANITIZER_COMMON_CFLAGS)
231228
append_list_if(COMPILER_RT_HAS_WD4722_FLAG /wd4722 SANITIZER_COMMON_CFLAGS)
232229
append_list_if(COMPILER_RT_HAS_WD4800_FLAG /wd4800 SANITIZER_COMMON_CFLAGS)
233230

234-
# Set common link flags.
235-
append_list_if(COMPILER_RT_HAS_NODEFAULTLIBS_FLAG -nodefaultlibs SANITIZER_COMMON_LINK_FLAGS)
236-
237-
if (SANITIZER_USE_COMPILER_RT)
238-
list(APPEND SANITIZER_COMMON_LINK_FLAGS -rtlib=compiler-rt)
239-
find_compiler_rt_library(builtins COMPILER_RT_BUILTINS_LIBRARY)
240-
list(APPEND SANITIZER_COMMON_LINK_LIBS ${COMPILER_RT_BUILTINS_LIBRARY})
241-
else()
242-
append_list_if(COMPILER_RT_HAS_GCC_S_LIB gcc_s SANITIZER_COMMON_LINK_LIBS)
243-
endif()
244-
245-
append_list_if(COMPILER_RT_HAS_LIBC c SANITIZER_COMMON_LINK_LIBS)
246-
247231
# Warnings to turn off for all libraries, not just sanitizers.
248232
append_string_if(COMPILER_RT_HAS_WUNUSED_PARAMETER_FLAG -Wno-unused-parameter CMAKE_C_FLAGS CMAKE_CXX_FLAGS)
249233

cmake/Modules/HandleCompilerRT.cmake

Lines changed: 0 additions & 21 deletions
This file was deleted.

cmake/config-ix.cmake

Lines changed: 1 addition & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
include(CMakePushCheckState)
2-
include(CheckCCompilerFlag)
32
include(CheckCXXCompilerFlag)
43
include(CheckLibraryExists)
54
include(CheckSymbolExists)
@@ -12,26 +11,6 @@ function(check_linker_flag flag out_var)
1211
cmake_pop_check_state()
1312
endfunction()
1413

15-
check_library_exists(c fopen "" COMPILER_RT_HAS_LIBC)
16-
if (NOT SANITIZER_USE_COMPILER_RT)
17-
check_library_exists(gcc_s __gcc_personality_v0 "" COMPILER_RT_HAS_GCC_S_LIB)
18-
endif()
19-
20-
check_c_compiler_flag(-nodefaultlibs COMPILER_RT_HAS_NODEFAULTLIBS_FLAG)
21-
if (COMPILER_RT_HAS_NODEFAULTLIBS_FLAG)
22-
set(CMAKE_REQUIRED_FLAGS "${CMAKE_REQUIRED_FLAGS} -nodefaultlibs")
23-
if (COMPILER_RT_HAS_LIBC)
24-
list(APPEND CMAKE_REQUIRED_LIBRARIES c)
25-
endif ()
26-
if (SANITIZER_USE_COMPILER_RT)
27-
list(APPEND CMAKE_REQUIRED_FLAGS -rtlib=compiler-rt)
28-
find_compiler_rt_library(builtins COMPILER_RT_BUILTINS_LIBRARY)
29-
list(APPEND CMAKE_REQUIRED_LIBRARIES "${COMPILER_RT_BUILTINS_LIBRARY}")
30-
elseif (COMPILER_RT_HAS_GCC_S_LIB)
31-
list(APPEND CMAKE_REQUIRED_LIBRARIES gcc_s)
32-
endif ()
33-
endif ()
34-
3514
# CodeGen options.
3615
check_cxx_compiler_flag(-fPIC COMPILER_RT_HAS_FPIC_FLAG)
3716
check_cxx_compiler_flag(-fPIE COMPILER_RT_HAS_FPIE_FLAG)
@@ -94,6 +73,7 @@ check_cxx_compiler_flag(/wd4800 COMPILER_RT_HAS_WD4800_FLAG)
9473
check_symbol_exists(__func__ "" COMPILER_RT_HAS_FUNC_SYMBOL)
9574

9675
# Libraries.
76+
check_library_exists(c fopen "" COMPILER_RT_HAS_LIBC)
9777
check_library_exists(dl dlopen "" COMPILER_RT_HAS_LIBDL)
9878
check_library_exists(rt shm_open "" COMPILER_RT_HAS_LIBRT)
9979
check_library_exists(m pow "" COMPILER_RT_HAS_LIBM)

lib/asan/CMakeLists.txt

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ set(ASAN_CFLAGS ${SANITIZER_COMMON_CFLAGS})
4040

4141
append_rtti_flag(OFF ASAN_CFLAGS)
4242

43-
set(ASAN_DYNAMIC_LINK_FLAGS ${SANITIZER_COMMON_LINK_FLAGS})
43+
set(ASAN_DYNAMIC_LINK_FLAGS)
4444

4545
if(ANDROID)
4646
# On Android, -z global does not do what it is documented to do.
@@ -65,8 +65,7 @@ append_list_if(COMPILER_RT_HAS_FTLS_MODEL_INITIAL_EXEC
6565
-ftls-model=initial-exec ASAN_DYNAMIC_CFLAGS)
6666
append_list_if(MSVC /DEBUG ASAN_DYNAMIC_LINK_FLAGS)
6767

68-
set(ASAN_DYNAMIC_LIBS ${SANITIZER_COMMON_LINK_LIBS})
69-
68+
append_list_if(COMPILER_RT_HAS_LIBC c ASAN_DYNAMIC_LIBS)
7069
append_list_if(COMPILER_RT_HAS_LIBDL dl ASAN_DYNAMIC_LIBS)
7170
append_list_if(COMPILER_RT_HAS_LIBRT rt ASAN_DYNAMIC_LIBS)
7271
append_list_if(COMPILER_RT_HAS_LIBM m ASAN_DYNAMIC_LIBS)

lib/lsan/CMakeLists.txt

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,8 +29,6 @@ add_compiler_rt_object_libraries(RTLSanCommon
2929
if(COMPILER_RT_HAS_LSAN)
3030
add_compiler_rt_component(lsan)
3131
if(APPLE)
32-
set(LSAN_LINK_LIBS ${SANITIZER_COMMON_LINK_LIBS})
33-
3432
add_weak_symbols("lsan" WEAK_SYMBOL_LINK_FLAGS)
3533
add_weak_symbols("sanitizer_common" WEAK_SYMBOL_LINK_FLAGS)
3634

@@ -44,8 +42,7 @@ if(COMPILER_RT_HAS_LSAN)
4442
RTSanitizerCommon
4543
RTSanitizerCommonLibc
4644
CFLAGS ${LSAN_CFLAGS}
47-
LINK_FLAGS ${SANITIZER_COMMON_LINK_FLAGS} ${WEAK_SYMBOL_LINK_FLAGS}
48-
LINK_LIBS ${LSAN_LINK_LIBS}
45+
LINK_FLAGS ${WEAK_SYMBOL_LINK_FLAGS}
4946
PARENT_TARGET lsan)
5047
else()
5148
foreach(arch ${LSAN_SUPPORTED_ARCH})

lib/stats/CMakeLists.txt

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,6 @@ set_target_properties(stats PROPERTIES FOLDER "Compiler-RT Misc")
66
if(APPLE)
77
set(STATS_LIB_FLAVOR SHARED)
88

9-
set(STATS_LINK_LIBS ${SANITIZER_COMMON_LINK_LIBS})
10-
119
add_weak_symbols("asan" WEAK_SYMBOL_LINK_FLAGS)
1210
add_weak_symbols("ubsan" WEAK_SYMBOL_LINK_FLAGS)
1311
add_weak_symbols("sanitizer_common" WEAK_SYMBOL_LINK_FLAGS)
@@ -25,8 +23,7 @@ add_compiler_rt_runtime(clang_rt.stats
2523
OBJECT_LIBS RTSanitizerCommon
2624
RTSanitizerCommonLibc
2725
CFLAGS ${SANITIZER_COMMON_CFLAGS}
28-
LINK_FLAGS ${SANITIZER_COMMON_LINK_FLAGS} ${WEAK_SYMBOL_LINK_FLAGS}
29-
LINK_LIBS ${STATS_LINK_LIBS}
26+
LINK_FLAGS ${WEAK_SYMBOL_LINK_FLAGS}
3027
PARENT_TARGET stats)
3128

3229
add_compiler_rt_runtime(clang_rt.stats_client

lib/tsan/CMakeLists.txt

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -109,8 +109,6 @@ if(APPLE)
109109
set_source_files_properties(${TSAN_ASM_SOURCES} PROPERTIES LANGUAGE C)
110110
endif()
111111

112-
set(TSAN_LINK_LIBS ${SANITIZER_COMMON_LINK_LIBS})
113-
114112
add_weak_symbols("ubsan" WEAK_SYMBOL_LINK_FLAGS)
115113
add_weak_symbols("sanitizer_common" WEAK_SYMBOL_LINK_FLAGS)
116114

@@ -124,8 +122,7 @@ if(APPLE)
124122
RTSanitizerCommonLibc
125123
RTUbsan
126124
CFLAGS ${TSAN_RTL_CFLAGS}
127-
LINK_FLAGS ${SANITIZER_COMMON_LINK_FLAGS} ${WEAK_SYMBOL_LINK_FLAGS}
128-
LINK_LIBS ${TSAN_LINK_LIBS}
125+
LINK_FLAGS ${WEAK_SYMBOL_LINK_FLAGS}
129126
PARENT_TARGET tsan)
130127
add_compiler_rt_object_libraries(RTTsan_dynamic
131128
OS ${TSAN_SUPPORTED_OS}

lib/tsan/dd/CMakeLists.txt

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,10 @@ set(DD_SOURCES
1010
dd_interceptors.cc
1111
)
1212

13-
set(DD_LINKLIBS ${SANITIZER_COMMON_LINK_LIBS})
14-
13+
set(DD_LINKLIBS)
1514
append_list_if(COMPILER_RT_HAS_LIBDL dl DD_LINKLIBS)
1615
append_list_if(COMPILER_RT_HAS_LIBRT rt DD_LINKLIBS)
1716
append_list_if(COMPILER_RT_HAS_LIBPTHREAD pthread DD_LINKLIBS)
18-
append_list_if(COMPILER_RT_HAS_LIBSTDCXX stdc++ DD_LINKLIBS)
1917

2018
add_custom_target(dd)
2119
# Deadlock detector is currently supported on 64-bit Linux only.
@@ -42,7 +40,6 @@ if(CAN_TARGET_x86_64 AND UNIX AND NOT APPLE AND NOT ANDROID)
4240
$<TARGET_OBJECTS:RTInterception.${arch}>
4341
$<TARGET_OBJECTS:RTSanitizerCommon.${arch}>
4442
$<TARGET_OBJECTS:RTSanitizerCommonLibc.${arch}>
45-
LINK_FLAGS ${SANITIZER_COMMON_LINK_FLAGS}
4643
LINK_LIBS ${DD_LINKLIBS}
4744
PARENT_TARGET dd)
4845
endif()

lib/ubsan/CMakeLists.txt

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -34,12 +34,9 @@ set(UBSAN_CXXFLAGS ${SANITIZER_COMMON_CFLAGS})
3434
append_rtti_flag(ON UBSAN_CXXFLAGS)
3535
append_list_if(SANITIZER_CAN_USE_CXXABI -DUBSAN_CAN_USE_CXXABI UBSAN_CXXFLAGS)
3636

37-
set(UBSAN_DYNAMIC_LIBS ${SANITIZER_COMMON_LINK_LIBS})
38-
3937
append_list_if(COMPILER_RT_HAS_LIBDL dl UBSAN_DYNAMIC_LIBS)
4038
append_list_if(COMPILER_RT_HAS_LIBRT rt UBSAN_DYNAMIC_LIBS)
4139
append_list_if(COMPILER_RT_HAS_LIBPTHREAD pthread UBSAN_DYNAMIC_LIBS)
42-
append_list_if(COMPILER_RT_HAS_LIBSTDCXX stdc++ UBSAN_DYNAMIC_LIBS)
4340

4441
add_compiler_rt_component(ubsan)
4542

@@ -158,7 +155,6 @@ else()
158155
RTSanitizerCommonLibc
159156
RTUbsan
160157
CFLAGS ${UBSAN_CFLAGS}
161-
LINK_FLAGS ${SANITIZER_COMMON_LINK_FLAGS}
162158
LINK_LIBS ${UBSAN_DYNAMIC_LIBS}
163159
PARENT_TARGET ubsan)
164160

@@ -170,7 +166,6 @@ else()
170166
RTUbsan
171167
RTUbsan_cxx
172168
CFLAGS ${UBSAN_CXXFLAGS}
173-
LINK_FLAGS ${SANITIZER_COMMON_LINK_FLAGS}
174169
LINK_LIBS ${UBSAN_DYNAMIC_LIBS}
175170
PARENT_TARGET ubsan)
176171

0 commit comments

Comments
 (0)