[PATCH] D27993: [libFuzzer] Diff 44 - Update cmake for libFuzzer. Remove requiring LLVM_USE_SANITIZE_COVERAGE .

Kostya Serebryany via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 4 18:25:15 PST 2017


kcc added a comment.

In https://reviews.llvm.org/D27993#636374, @mpividori wrote:

> @kcc We can continue using `LLVM_USE_SANITIZE_COVERAGE` for the rest of the repository, when we need instrumentation in all the code.
>  But for building `libFuzzer`, we don't need that, so I think it makes sense to use a different flag to enable/disable libFuzzer. 
>  So, I propose to use `ENABLE_LIBFUZZER` in this Cmake file, instead of `LLVM_USE_SANITIZER`.
>  Would you agree?
>  Thanks,


It means one more non-default build config... No, I'd prefer to avoid it.


Repository:
  rL LLVM

https://reviews.llvm.org/D27993





More information about the llvm-commits mailing list