diff options
| author | Alexander Richardson <arichardson.kde@gmail.com> | 2019-07-27 12:30:15 +0000 |
|---|---|---|
| committer | Alexander Richardson <arichardson.kde@gmail.com> | 2019-07-27 12:30:15 +0000 |
| commit | 51bfb84852e5c3bf746a34a455504c70540a55c3 (patch) | |
| tree | f5a702d4fb68b1b90835e2cece67f9b5ff37af2f /llvm/tools/llvm-strings/llvm-strings.cpp | |
| parent | 3ff6126487ddc7636af83c70ee11a586d83ed59a (diff) | |
| download | bcm5719-llvm-51bfb84852e5c3bf746a34a455504c70540a55c3.tar.gz bcm5719-llvm-51bfb84852e5c3bf746a34a455504c70540a55c3.zip | |
[compiler-rt] Fix running tests on macOS when XCode is not installed
Summary:
If XCode is not installed, `xcodebuild -version -sdk macosx Path` will give
xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance
In this case the variable OSX_SYSROOT will be empty and
OSX_SYSROOT_FLAG is set to "-isysroot" (without a path).
This then causes the CompilerRTUnitTestCheckCxx target failed to for me
because "${COMPILER_RT_TEST_COMPILER} ${OSX_SYSROOT_FLAG} -E" expanded to
"clang -isysroot -E". This results in a warning "sysroot -E does not exist"
and the target fails to run because the C++ headers cannot be found.
Reviewers: beanz, kubamracek
Reviewed By: beanz
Subscribers: dberris, mgorny, #sanitizers, llvm-commits
Tags: #sanitizers, #llvm
Differential Revision: https://reviews.llvm.org/D65323
llvm-svn: 367170
Diffstat (limited to 'llvm/tools/llvm-strings/llvm-strings.cpp')
0 files changed, 0 insertions, 0 deletions
