Add libc++ to link XRay test cases if libc++ is used to build CLANG
Summary: When libc++ is used to build CLANG, its XRay libraries libclang_rt.xray-*.a have dependencies on libc++. Therefore, libc++ is needed to link and run XRay test cases. For Linux -rpath is also needed to specify where to load libc++. This change sets macro LLVM_LIBCXX_USED to 1 if libc++ is actually used in the build. XRay tests then check the flag and add -L<llvm_shlib_dir> -lc++ and -Wl,-rpath=<llvm_shlib_dir> if needed. Reviewers: hubert.reinterpretcast, amyk, dberris, jasonliu, sfertile, EricWF Subscribers: dberris, mgorny, jsji, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D61016 llvm-svn: 360060
This commit is contained in:
parent
a64bd09ec4
commit
865a39d328
|
@ -49,6 +49,7 @@ else:
|
|||
set_default("target_suffix", "-%s" % config.target_arch)
|
||||
|
||||
set_default("have_zlib", "@HAVE_LIBZ@")
|
||||
set_default("libcxx_used", "@LLVM_LIBCXX_USED@")
|
||||
|
||||
# LLVM tools dir can be passed in lit parameters, so try to
|
||||
# apply substitution.
|
||||
|
|
|
@ -10,8 +10,16 @@ config.test_source_root = os.path.dirname(__file__)
|
|||
|
||||
# Setup default compiler flags use with -fxray-instrument option.
|
||||
clang_xray_cflags = (['-fxray-instrument', config.target_cflags])
|
||||
clang_xray_cxxflags = config.cxx_mode_flags + clang_xray_cflags
|
||||
|
||||
# If libc++ was used to build XRAY libraries, libc++ is needed. Fix applied
|
||||
# to Linux only since -rpath may not be portable. This can be extended to
|
||||
# other platforms.
|
||||
if config.libcxx_used == "1" and config.host_os == "Linux":
|
||||
clang_xray_cflags = clang_xray_cflags + (['-L%s -lc++ -Wl,-rpath=%s'
|
||||
% (config.llvm_shlib_dir,
|
||||
config.llvm_shlib_dir)])
|
||||
|
||||
clang_xray_cxxflags = config.cxx_mode_flags + clang_xray_cflags
|
||||
|
||||
def build_invocation(compile_flags):
|
||||
return ' ' + ' '.join([config.clang] + compile_flags) + ' '
|
||||
|
|
|
@ -14,6 +14,7 @@ if(NOT DEFINED LLVM_STDLIB_HANDLED)
|
|||
|
||||
include(CheckCXXCompilerFlag)
|
||||
include(CheckLinkerFlag)
|
||||
set(LLVM_LIBCXX_USED 0)
|
||||
if(LLVM_ENABLE_LIBCXX)
|
||||
if(LLVM_COMPILER_IS_GCC_COMPATIBLE)
|
||||
check_cxx_compiler_flag("-stdlib=libc++" CXX_COMPILER_SUPPORTS_STDLIB)
|
||||
|
@ -22,6 +23,7 @@ if(NOT DEFINED LLVM_STDLIB_HANDLED)
|
|||
append("-stdlib=libc++"
|
||||
CMAKE_CXX_FLAGS CMAKE_EXE_LINKER_FLAGS CMAKE_SHARED_LINKER_FLAGS
|
||||
CMAKE_MODULE_LINKER_FLAGS)
|
||||
set(LLVM_LIBCXX_USED 1)
|
||||
else()
|
||||
message(WARNING "Can't specify libc++ with '-stdlib='")
|
||||
endif()
|
||||
|
|
Loading…
Reference in New Issue