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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to