Details
Diff Detail
- Build Status
Buildable 11111 Build 11111: arc lint + arc unit
Event Timeline
llvm/cmake/modules/AddLLVM.cmake | ||
---|---|---|
897–902 | How / where does this get set? |
llvm/cmake/modules/AddLLVM.cmake | ||
---|---|---|
897–902 | In the CMake configure command. See https://github.com/google/oss-fuzz/pull/885. |
llvm/cmake/modules/AddLLVM.cmake | ||
---|---|---|
897–902 | Okay, if we're going to use a cmake variable for this we should do it right and define a cache variable with documentation in the top level CMakeLists.txt file. I'd probably add it next to where we add LLVM_USE_SANITIZER. Also, we should namespace this appropriately. Something like LLVM_LIB_FUZZING_ENGINE or LLVM_FUZZING_ENGINE. Optionally, we could also set it up so that we default that to libFuzzer in the LLVM_USE_SANITIZE_COVERAGE case so that we don't even need two cases here. |
llvm/cmake/modules/AddLLVM.cmake | ||
---|---|---|
897–902 | Regarding switching the configuration under LLVM_USE_SANITIZE_COVERAGE, I don't think we want to do that. OSS-Fuzz uses its own instrumentation which we don't want to mix with -fsanitize=fuzzer. But having -fsanitize=fuzzer by default seems helpful for developers. |
llvm/cmake/modules/AddLLVM.cmake | ||
---|---|---|
897–902 | Right. |
How / where does this get set?