Add an option to statically link a custom libc++.

MSan and TSan both require instrumenting everything. Add some machinery so we
can do this on the bots.

Change-Id: I7d2106bc852ee976455d18787d3a20a35373a9e7
Reviewed-on: https://boringssl-review.googlesource.com/30644
Reviewed-by: Adam Langley <agl@google.com>
This commit is contained in:
David Benjamin 2018-08-09 15:33:07 -05:00 committed by Adam Langley
parent 1f0d54b8a1
commit e9ae99bba2
4 changed files with 78 additions and 0 deletions

2
.gitignore vendored
View File

@ -15,6 +15,8 @@ util/bot/cmake-win32
util/bot/cmake-win32.zip util/bot/cmake-win32.zip
util/bot/golang util/bot/golang
util/bot/gyp util/bot/gyp
util/bot/libcxx
util/bot/libcxxabi
util/bot/libFuzzer util/bot/libFuzzer
util/bot/llvm-build util/bot/llvm-build
util/bot/nasm-win32.exe util/bot/nasm-win32.exe

View File

@ -38,6 +38,9 @@ if (NOT GO_EXECUTABLE)
message(FATAL_ERROR "Could not find Go") message(FATAL_ERROR "Could not find Go")
endif() endif()
if (USE_CUSTOM_LIBCXX)
set(BORINGSSL_ALLOW_CXX_RUNTIME 1)
endif()
if (BORINGSSL_ALLOW_CXX_RUNTIME) if (BORINGSSL_ALLOW_CXX_RUNTIME)
add_definitions(-DBORINGSSL_ALLOW_CXX_RUNTIME) add_definitions(-DBORINGSSL_ALLOW_CXX_RUNTIME)
endif() endif()
@ -361,6 +364,61 @@ if (${ARCH} STREQUAL "x86" AND APPLE AND ${CMAKE_VERSION} VERSION_LESS "3.0")
set(ARCH "x86_64") set(ARCH "x86_64")
endif() endif()
if (USE_CUSTOM_LIBCXX)
if (NOT CLANG)
message(FATAL_ERROR "USE_CUSTOM_LIBCXX only supported with Clang")
endif()
# CMAKE_CXX_FLAGS ends up in the linker flags as well, so use
# add_compile_options. There does not appear to be a way to set
# language-specific compile-only flags.
add_compile_options("-nostdinc++")
set(CMAKE_CXX_LINK_FLAGS "${CMAKE_CXX_LINK_FLAGS} -nostdlib++")
include_directories(
SYSTEM
util/bot/libcxx/include
util/bot/libcxxabi/include
)
# This is patterned after buildtools/third_party/libc++/BUILD.gn and
# buildtools/third_party/libc++abi/BUILD.gn in Chromium.
file(GLOB LIBCXX_SOURCES "util/bot/libcxx/src/*.cpp")
file(GLOB LIBCXXABI_SOURCES "util/bot/libcxxabi/src/*.cpp")
# This file is meant for exception-less builds.
list(REMOVE_ITEM LIBCXXABI_SOURCES "trunk/src/cxa_noexception.cpp")
# libc++ also defines new and delete.
list(REMOVE_ITEM LIBCXXABI_SOURCES "trunk/src/stdlib_new_delete.cpp")
if (TSAN)
# ThreadSanitizer tries to intercept these symbols. Skip them to avoid
# symbol conflicts.
list(REMOVE_ITEM LIBCXXABI_SOURCES "trunk/src/cxa_guard.cpp")
endif()
add_library(libcxxabi ${LIBCXXABI_SOURCES})
target_compile_definitions(
libcxxabi PRIVATE
-D_LIBCPP_ENABLE_CXX17_REMOVED_UNEXPECTED_FUNCTIONS
)
set_target_properties(libcxxabi PROPERTIES COMPILE_FLAGS "-Wno-missing-prototypes -Wno-implicit-fallthrough")
add_library(libcxx ${LIBCXX_SOURCES})
if (ASAN OR MSAN OR TSAN)
# Sanitizers try to intercept new and delete.
target_compile_definitions(
libcxx PRIVATE
-D_LIBCPP_DISABLE_NEW_DELETE_DEFINITIONS
)
endif()
target_compile_definitions(
libcxx PRIVATE
-D_LIBCPP_BUILDING_LIBRARY
-DLIBCXX_BUILDING_LIBCXXABI
)
target_link_libraries(libcxx libcxxabi)
endif()
# Add minimal googletest targets. The provided one has many side-effects, and # Add minimal googletest targets. The provided one has many side-effects, and
# googletest has a very straightforward build. # googletest has a very straightforward build.
add_library(boringssl_gtest third_party/googletest/src/gtest-all.cc) add_library(boringssl_gtest third_party/googletest/src/gtest-all.cc)

View File

@ -216,6 +216,12 @@ if(NOT MSVC AND NOT ANDROID)
target_link_libraries(crypto pthread) target_link_libraries(crypto pthread)
endif() endif()
# Every target depends on crypto, so we add libcxx as a dependency here to
# simplify injecting it everywhere.
if(USE_CUSTOM_LIBCXX)
target_link_libraries(crypto libcxx)
endif()
# TODO(davidben): Convert the remaining tests to GTest. # TODO(davidben): Convert the remaining tests to GTest.
add_executable( add_executable(
crypto_test crypto_test

View File

@ -19,6 +19,7 @@ vars = {
'checkout_fuzzer': False, 'checkout_fuzzer': False,
'checkout_sde': False, 'checkout_sde': False,
'checkout_nasm': False, 'checkout_nasm': False,
'checkout_libcxx': False,
} }
deps = { deps = {
@ -39,6 +40,17 @@ deps = {
'url': Var('chromium_git') + '/chromium/llvm-project/compiler-rt/lib/fuzzer.git' + '@' + '658ff786a213703ff0df6ba4a288e9a1e218c074', 'url': Var('chromium_git') + '/chromium/llvm-project/compiler-rt/lib/fuzzer.git' + '@' + '658ff786a213703ff0df6ba4a288e9a1e218c074',
'condition': 'checkout_fuzzer', 'condition': 'checkout_fuzzer',
}, },
# Update the following revisions from
# https://chromium.googlesource.com/chromium/buildtools/+/master/DEPS
'boringssl/util/bot/libcxx': {
'url': Var('chromium_git') + '/chromium/llvm-project/libcxx.git' + '@' + '85a7702b4cc5d69402791fe685f151cf3076be71',
'condition': 'checkout_libcxx',
},
'boringssl/util/bot/libcxxabi': {
'url': Var('chromium_git') + '/chromium/llvm-project/libcxxabi.git' + '@' + '05a73941f3fb177c4a891d4ff2a4ed5785e3b80c',
'condition': 'checkout_libcxx',
},
} }
recursedeps = [ recursedeps = [