[PATCH] D31614: Simplify test a bit

[Rafael Ávila de Espíndola via Phabricator via llvm-commits]

rafael created this revision.
Herald added a subscriber: kubamracek.

There are two cases to consider:

We are using the internal shell. This will still fail because of ulimit.
We are using an external shell. In this case the difference is that we now also constrain FileCheck to use less than 4 MB of of stack, which it should :-)


https://reviews.llvm.org/D31614

Files:
  compiler-rt/test/asan/TestCases/Posix/deep_call_stack.cc


Index: compiler-rt/test/asan/TestCases/Posix/deep_call_stack.cc
===================================================================
--- compiler-rt/test/asan/TestCases/Posix/deep_call_stack.cc
+++ compiler-rt/test/asan/TestCases/Posix/deep_call_stack.cc
@@ -1,6 +1,7 @@
 // Check that UAR mode can handle very deep recusrion.
-// RUN: %clangxx_asan -O2 %s -o %t && \
-// RUN:   (ulimit -s 4096; %env_asan_opts=detect_stack_use_after_return=1 %run %t) 2>&1 | FileCheck %s
+// RUN: %clangxx_asan -O2 %s -o %t
+// RUN: ulimit -s 4096
+// RUN: %env_asan_opts=detect_stack_use_after_return=1 %run %t 2>&1 | FileCheck %s
 
 // Also check that use_sigaltstack+verbosity doesn't crash.
 // RUN: %env_asan_opts=verbosity=1:use_sigaltstack=1:detect_stack_use_after_return=1 %run %t  | FileCheck %s


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D31614.93907.patch
Type: text/x-patch
Size: 788 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170403/31166b5c/attachment.bin>