[PATCH] D20741: [LibFuzzer] Fix weak linking issues building tests under OSX.
Kostya Serebryany via llvm-commits
llvm-commits at lists.llvm.org
Fri May 27 16:05:10 PDT 2016
kcc added a comment.
In http://reviews.llvm.org/D20741#442890, @delcypher wrote:
> @kcc:
>
> > Let's try dlsym. Let's create a separate file for that.
>
>
> Do you also want to use the dlsym implementation on Linux? I'm imagining a struct containing function pointers to all the functions that were previously declared weak. We could have a different implementation for initializing those function pointers for different platforms or we could have OSX and Linux both share the same implementation. What would you prefer?
I expect dlsym to work equally well on linux, so, yes, let's make it the same on both.
+1 to struct of function pointers. Declare it in a separate header file, initialize it in a separate .cpp file.
FuzzerDlsym.cpp? FuzzerWeakSymbols.cpp? [naming is hard]
http://reviews.llvm.org/D20741
More information about the llvm-commits
mailing list