Skip to content

Commit 16c4456

Browse files
author
Eli Friedman
committedJun 14, 2018
[compiler-rt] [builtins] Don't build __atomic_* by default.
The locks need to be implemented in a shared library to work correctly, so they shouldn't be part of libclang_rt.builtins.a, except in specialized scenarios where the user can prove it will only be linked once. Differential Revision: https://reviews.llvm.org/D47606 llvm-svn: 334779
1 parent bc023c9 commit 16c4456

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed
 

‎compiler-rt/lib/builtins/CMakeLists.txt

+2-2
Original file line numberDiff line numberDiff line change
@@ -173,8 +173,8 @@ set(GENERIC_TF_SOURCES
173173
trunctfsf2.c)
174174

175175
option(COMPILER_RT_EXCLUDE_ATOMIC_BUILTIN
176-
"Skip the atomic builtin (this may be needed if system headers are unavailable)"
177-
Off)
176+
"Skip the atomic builtin (these should normally be provided by a shared library)"
177+
On)
178178

179179
if(NOT FUCHSIA AND NOT COMPILER_RT_BAREMETAL_BUILD)
180180
set(GENERIC_SOURCES

0 commit comments

Comments
 (0)