morehouse added inline comments.
================ Comment at: clang/tools/clang-fuzzer/ClangFuzzer.cpp:20 extern "C" int LLVMFuzzerTestOneInput(uint8_t *data, size_t size) { std::string s((const char *)data, size); ---------------- morehouse wrote: > vitalybuka wrote: > > Do we want replace this fuzzer? Why not just add another one? > The idea was to keep ClangFuzzer and add ClangProtoFuzzer alongside it. > However, the two share a fair amount of code, which was factored out into > HandleCXX. > It's ok to share code, but I don't see fuzzer with accept string as is. That's exactly what this fuzzer is doing. https://reviews.llvm.org/D36324 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits