[PATCH] D20977: [LibFuzzer] Split the fuzzer-oom.test into two tests.

Dan Liew via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 6 18:13:07 PDT 2016


delcypher added inline comments.

================
Comment at: lib/Fuzzer/test/fuzzer-oom-darwin.test:1
@@ +1,2 @@
+REQUIRES: darwin
+RUN: not LLVMFuzzer-OutOfMemoryTest -rss_limit_mb=10 2>&1 | FileCheck %s
----------------
kcc wrote:
> Why does this require darwin? 
Technically it doesn't but my assumption is this would be the test for darwin (e.g. OSX). Do you want me to remove the `REQUIRES: darwin` line and remove darwin from the name of the test?


http://reviews.llvm.org/D20977





More information about the llvm-commits mailing list