[PATCH] D29420: [libFuzzer] Isolate merge tests that require posix.

Kostya Serebryany via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 1 13:44:46 PST 2017


kcc accepted this revision.
kcc added a comment.
This revision is now accepted and ready to land.

LGTM with two nits



================
Comment at: lib/Fuzzer/test/merge-posix.test:1
+CHECK: BINGO
+
----------------
you don't need this line


================
Comment at: lib/Fuzzer/test/merge-posix.test:17
+
+REQUIRES: posix
----------------
put REQUIRES at the top


https://reviews.llvm.org/D29420





More information about the llvm-commits mailing list