[PATCH] D56194: [gn build] Add fuzzers in llvm/tools that are needed for check-llvm

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 1 19:52:09 PST 2019


phosek added inline comments.


================
Comment at: llvm/utils/gn/build/fuzzer.gni:29
+template("fuzzer") {
+  assert(defined(invoker.dummy_main))
+  assert(defined(invoker.sources))
----------------
I checked the LLVM `add_llvm_fuzzer` function and `DUMMY_MAIN` is just an optional argument which is only included in `LLVM_OPTIONAL_SOURCES` for IDE but otherwise ignored if `llvm_use_sanitize_coverage` or `llvm_use_sanitize_coverage` is set. Shouldn't this behave the same?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56194/new/

https://reviews.llvm.org/D56194





More information about the llvm-commits mailing list