[PATCH] D38642: [clang-fuzzer] Allow building without coverage instrumentation.

Kostya Serebryany via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 6 17:02:31 PDT 2017


kcc added a comment.

In https://reviews.llvm.org/D38642#891125, @morehouse wrote:

> In https://reviews.llvm.org/D38642#891074, @kcc wrote:
>
> > If you can *easily* share main() with the one in LLVM -- do it, otherwise don't bother.
>
>
> Does the fuzzer main come from LLVM or compiler-rt now?  There's still FuzzerMain.cpp in LLVM, but I'm not sure if we should be using that or not.


Don't reuse FuzzerMain.cpp. 
There is llvm/tools/llvm-isel-fuzzer/DummyISelFuzzer.cpp which your main() duplicates, but these are in different projects (llvm vs clang) so perhaps it's ok.


https://reviews.llvm.org/D38642





More information about the cfe-commits mailing list