[PATCH] D27729: [libFuzzer] Diff 20 - Update tests to use more general functions instead of posix specific.

Marcos Pividori via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 14 13:09:58 PST 2016


mpividori added inline comments.


================
Comment at: lib/Fuzzer/test/OutOfMemoryTest.cpp:5
 // Tests OOM handling.
+#include "FuzzerUtil.h"
 #include <assert.h>
----------------
mpividori wrote:
> kcc wrote:
> > Please don't. 
> > I want the tests to remain free of any such deps. 
> Free from dependencies from libFuzzer?? Even if this tests are linked with the LLVMFuzzer.lib library? You want me to remove the include and declare the function signature? I don't understand...
In `CustomCrossOverTest.cpp`, `CustomMutatorTest.cpp`, and other tests, you also include:
 ` #include "FuzzerInterface.h" ` ...


Repository:
  rL LLVM

https://reviews.llvm.org/D27729





More information about the llvm-commits mailing list