[llvm-commits] [test-suite] r58063 - /test-suite/trunk/MultiSource/Applications/ClamAV/clamscan_clamscan.c

Dan Gohman gohman at apple.com
Thu Oct 23 18:47:08 PDT 2008


Author: djg
Date: Thu Oct 23 20:47:08 2008
New Revision: 58063

URL: http://llvm.org/viewvc/llvm-project?rev=58063&view=rev
Log:
Disable the call to setrlimit (which only happened when C_LINUX)
is defined. There's no need to override the core-file settings
in the context of the LLVM test-suite. And if the setrlimit fails,
a message was printed, leading to spurious test failures.

Modified:
    test-suite/trunk/MultiSource/Applications/ClamAV/clamscan_clamscan.c

Modified: test-suite/trunk/MultiSource/Applications/ClamAV/clamscan_clamscan.c
URL: http://llvm.org/viewvc/llvm-project/test-suite/trunk/MultiSource/Applications/ClamAV/clamscan_clamscan.c?rev=58063&r1=58062&r2=58063&view=diff

==============================================================================
--- test-suite/trunk/MultiSource/Applications/ClamAV/clamscan_clamscan.c (original)
+++ test-suite/trunk/MultiSource/Applications/ClamAV/clamscan_clamscan.c Thu Oct 23 20:47:08 2008
@@ -92,6 +92,9 @@
 
 
     if(opt_check(opt, "debug")) {
+#if 0 // In the context of the LLVM testsuite, we don't want to override
+      // the setrlimit values. Among other problems, it can fail, leading
+      // to spurious diffs.
 #if defined(C_LINUX)
 	    /* njh at bandsman.co.uk: create a dump if needed */
 	    struct rlimit rlim;
@@ -100,6 +103,7 @@
 	if(setrlimit(RLIMIT_CORE, &rlim) < 0)
 	    perror("setrlimit");
 #endif
+#endif
 	cl_debug(); /* enable debug messages */
     }
 





More information about the llvm-commits mailing list