[PATCH] D49526: Updated llvm-proto-fuzzer to execute the compiled code
Matt Morehouse via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 19 15:20:51 PDT 2018
morehouse added inline comments.
================
Comment at: clang/tools/clang-fuzzer/handle-llvm/CMakeLists.txt:21
+ set(handle_llvm_deps)
+endif()
----------------
How are you doing your diff? Some of these changes are already upstream. Please rebase
================
Comment at: clang/tools/clang-fuzzer/handle-llvm/fuzzer_initialize.cpp:52
+ return 0;
+}
----------------
Can we use `fuzzer-initialize/` now?
================
Comment at: clang/tools/clang-fuzzer/handle-llvm/handle_llvm.cpp:54
+
+static std::string OptIR;
+
----------------
Unless there's a good reason for global, let's make this a local.
Repository:
rC Clang
https://reviews.llvm.org/D49526
More information about the cfe-commits
mailing list