This is an archive of the discontinued LLVM Phabricator instance.

Add lit-style tests for the Fuzzer library
ClosedPublic

Authored by kcc on Jan 27 2015, 5:54 PM.

Diff Detail

Event Timeline

kcc updated this revision to Diff 18872.Jan 27 2015, 5:54 PM
kcc retitled this revision from to Add lit-style tests for the Fuzzer library.
kcc updated this object.
kcc edited the test plan for this revision. (Show Details)
kcc added a reviewer: samsonov.
kcc added a subscriber: Unknown Object (MLST).
samsonov accepted this revision.Jan 28 2015, 2:33 PM
samsonov edited edge metadata.

LGTM, once you address the comments. Please watch the bots after commit.

lib/Fuzzer/CMakeLists.txt
11

Put this under

if (LLVM_INCLUDE_TESTS)
lib/Fuzzer/test/fuzzer.test
1

delete empty line

12

What about ExactTest?

14

Do you need "-" at the end of this line?

lib/Fuzzer/test/lit.cfg
1

Delete this line.

8

empty line?

This revision is now accepted and ready to land.Jan 28 2015, 2:33 PM
kcc added inline comments.Jan 28 2015, 2:50 PM
lib/Fuzzer/CMakeLists.txt
11

done

lib/Fuzzer/test/fuzzer.test
1

done

12

Hah, good catch.
In the current state the Fuzzer is not capable of solving ExactTest (nor is AFL, for example).
I have a plan of attack though.
http://reviews.llvm.org/rL227387 is part of that.

14

It's a part of output, so yes.

lib/Fuzzer/test/lit.cfg
1

done

8

done

kcc closed this revision.Jan 28 2015, 2:50 PM