[PATCH] D38642: [clang-fuzzer] Allow building without coverage instrumentation.
Matt Morehouse via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 6 13:22:19 PDT 2017
morehouse added a comment.
In https://reviews.llvm.org/D38642#890963, @kcc wrote:
> It's not about coverage instrumentation (not) being present, but about libFuzzer's main() being present, right?
Yes.
> Will we be able to reuse some of Justin's code instead of creating one more main() function?
This reuses the code that Justin moved to FuzzMutate/FuzzerCLI. That's why the main is so short. But perhaps we could move the main itself into FuzzerCLI?
> Or, why not link with libFuzzer (-fsanitize=fuzzer at link time) even if we don't us einstrumentation at compile time?
When I tried this, I got undefined references to all kinds of `__sanitizer_cov_*` symbols.
https://reviews.llvm.org/D38642
More information about the cfe-commits
mailing list