[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