[PATCH] [fuzzer] Add a gtest-style test

Alexey Samsonov vonosmas at gmail.com
Fri Jan 30 12:02:56 PST 2015


================
Comment at: lib/Fuzzer/test/CMakeLists.txt:51
@@ -29,1 +50,3 @@
+
+set_target_properties(${TestBinaries} LLVMFuzzer-Unittest
   PROPERTIES RUNTIME_OUTPUT_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}
----------------
Looks like it's easier to add LLVMFuzzer-Unittest to ${TestBinaries}

================
Comment at: lib/Fuzzer/test/FuzzerUnittest.cpp:6
@@ +5,3 @@
+// Just to make it link.
+extern "C" void TestOneInput(const uint8_t *Data, size_t Size) {
+  abort();
----------------
Will this eventually go away? Shouldn't you layout LLVMFuzzer differently?

================
Comment at: lib/Fuzzer/test/FuzzerUnittest.cpp:60
@@ +59,3 @@
+        ExpectedUnitsWitThisLength.insert(U);
+    // std::cerr << "XXX " << FoundUnits.size() << " "
+    //          << ExpectedUnitsWitThisLength.size() << "\n";
----------------
Remove debugging output

http://reviews.llvm.org/D7287

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list