-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix installation on Clang-15 #37
Conversation
1. Fix finding of libFuzzer static library. 2. Fix finding of ASan and UBSan static libraries.
setup_utils/find_libfuzzer.sh
Outdated
@@ -24,11 +24,11 @@ if [[ "$uname" == "Darwin" ]]; then | |||
elif [[ "$uname" == "Linux" ]]; then | |||
machine="$(uname -m)" | |||
if [[ "$machine" == "x86_64" ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note: I haven't fixed OSX because I have no way to test this or determine the triple correctly.
setup_utils/find_libfuzzer.sh
Outdated
@@ -24,11 +24,11 @@ if [[ "$uname" == "Darwin" ]]; then | |||
elif [[ "$uname" == "Linux" ]]; then | |||
machine="$(uname -m)" | |||
if [[ "$machine" == "x86_64" ]]; then | |||
libpath="lib/linux/libclang_rt.fuzzer_no_main-x86_64.a" | |||
libpath="lib/x86_64-unknown-linux-gnu/libclang_rt.fuzzer_no_main.a" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I frankly don't need the changes in this file for OSS-Fuzz, I can hack around this files problems with LIBFUZZER_LIB. I'm happy to undo my changes to this file and let you figure out how to handle the differences between clang versions.
This will break builds with earlier versions of LLVM. Per discussion offline, Jonathan will post a partial change that allows clang-15 to work with LIBFUZZER_LIB. We'll make it automatic at a later point. |
Fix finding of ASan and UBSan static libraries.
Fixes: #36