[PATCH] D48106: implemented proto to llvm
Emmett Neyman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 19 11:43:17 PDT 2018
emmettneyman added inline comments.
================
Comment at: tools/clang-fuzzer/handle-llvm/\:62
+ initializeScavengerTestPass(*Registry);
+
+}
----------------
morehouse wrote:
> Does this initialization need to happen every time the fuzzer generates a new input, or can we call this from `LLVMFuzzerInitialize()` instead?
I was following the pattern from `handle_cxx.cpp` in which the initialization calls were made every time a new input was generated. However looking at the documentation and at `llvm-isel-fuzzer.cpp`, it seems like putting it in `LLVMFuzzerInitialize()` makes more sense. I will make this change in the next commit.
Repository:
rC Clang
https://reviews.llvm.org/D48106
More information about the cfe-commits
mailing list