diff options
| author | Max Moroz <mmoroz@chromium.org> | 2017-09-12 15:02:10 +0000 |
|---|---|---|
| committer | Max Moroz <mmoroz@chromium.org> | 2017-09-12 15:02:10 +0000 |
| commit | 1a33da240133e13a7afe77de0a595d82d06a8b4b (patch) | |
| tree | d4f2192e0c96337f4e193edb144c8b9b12222b6a /llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp | |
| parent | 0af5a772e06bf0060ee831faf2359a7f3600773d (diff) | |
| download | bcm5719-llvm-1a33da240133e13a7afe77de0a595d82d06a8b4b.tar.gz bcm5719-llvm-1a33da240133e13a7afe77de0a595d82d06a8b4b.zip | |
[libFuzzer] Fix lit files to make running tests more straightforward on Mac OS.
Summary:
Current implementation does not work if CMAKE_OSX_SYSROOT is not specified.
It silently generates invalid command with the following flags:
`-std=c++11 -lc++ -gline-tables-only -isysroot -fsanitize=address,fuzzer`
and then fails with the following error:
```
warning: no such sysroot directory: '-fsanitize=address,fuzzer' [-Wmissing-sysroot]"
<...>/RepeatedBytesTest.cpp:5:10: fatal error: 'assert.h' file not found
#include <assert.h>
^~~~~~~~~~
1 error generated.
```
However, if you have Command Line Tools installed, you have '/usr/include' dir.
In that case, it is not necessary to specify isysroot path.
Also, with the patch, in case of '/usr/include' does not exist, the '-sysroot'
path would be resolved automatically in compiler-rt/cmake/base-config-ix.cmake.
For more context, see the comment at `compiler-rt/cmake/base-config-ix.cmake#L76`
Reviewers: kcc, george.karpenkov
Reviewed By: kcc, george.karpenkov
Differential Revision: https://reviews.llvm.org/D37721
llvm-svn: 313033
Diffstat (limited to 'llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp')
0 files changed, 0 insertions, 0 deletions
