[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