[compiler-rt] 04dbb63 - [lsan][test] Enable many_tls_keys_pthread.cpp and disable swapcontext.cpp/fork_and_leak.cpp

[Fangrui Song via llvm-commits]

Author: Fangrui Song
Date: 2021-03-26T11:26:08-07:00
New Revision: 04dbb63400c5fa2f263d7473272509be572a367a

URL: https://github.com/llvm/llvm-project/commit/04dbb63400c5fa2f263d7473272509be572a367a
DIFF: https://github.com/llvm/llvm-project/commit/04dbb63400c5fa2f263d7473272509be572a367a.diff

LOG: [lsan][test] Enable many_tls_keys_pthread.cpp and disable swapcontext.cpp/fork_and_leak.cpp

With D98926, many_tls_keys_pthread.cpp appears to be working.

On glibc 2.30-0ubuntu2, swapcontext.cpp and Linux/fork_and_leak.cpp work fine
but they strangely fail on clang-cmake-aarch64-full
(https://lab.llvm.org/buildbot/#/builders/7/builds/2240).
Disable them for now.

Note: check-lsan was recently enabled on AArch64 in D98985. A test takes
10+ seconds. We should figure out the bottleneck.

Added: 
    

Modified: 
    compiler-rt/test/lsan/TestCases/Linux/fork_and_leak.cpp
    compiler-rt/test/lsan/TestCases/many_tls_keys_pthread.cpp
    compiler-rt/test/lsan/TestCases/swapcontext.cpp

Removed: 
    


################################################################################
diff  --git a/compiler-rt/test/lsan/TestCases/Linux/fork_and_leak.cpp b/compiler-rt/test/lsan/TestCases/Linux/fork_and_leak.cpp
index d7427ce3ed04..758cd819f16e 100644
--- a/compiler-rt/test/lsan/TestCases/Linux/fork_and_leak.cpp
+++ b/compiler-rt/test/lsan/TestCases/Linux/fork_and_leak.cpp
@@ -1,6 +1,9 @@
 // Test that leaks detected after forking without exec().
 // RUN: %clangxx_lsan %s -o %t && not %run %t 2>&1 | FileCheck %s
 
+/// Fails on clang-cmake-aarch64-full (glibc 2.27-3ubuntu1.4).
+// UNSUPPORTED: aarch64
+
 #include <assert.h>
 #include <stdlib.h>
 #include <sys/wait.h>

diff  --git a/compiler-rt/test/lsan/TestCases/many_tls_keys_pthread.cpp b/compiler-rt/test/lsan/TestCases/many_tls_keys_pthread.cpp
index 63a72481eca9..8c4970c9b137 100644
--- a/compiler-rt/test/lsan/TestCases/many_tls_keys_pthread.cpp
+++ b/compiler-rt/test/lsan/TestCases/many_tls_keys_pthread.cpp
@@ -7,7 +7,7 @@
 
 // On glibc, this requires the range returned by GetTLS to include
 // specific_1stblock and specific in `struct pthread`.
-// UNSUPPORTED: arm-linux, armhf-linux, aarch64
+// UNSUPPORTED: arm-linux, armhf-linux
 
 // TSD on NetBSD does not use TLS
 // UNSUPPORTED: netbsd

diff  --git a/compiler-rt/test/lsan/TestCases/swapcontext.cpp b/compiler-rt/test/lsan/TestCases/swapcontext.cpp
index d0999598ad4c..f78867cc0695 100644
--- a/compiler-rt/test/lsan/TestCases/swapcontext.cpp
+++ b/compiler-rt/test/lsan/TestCases/swapcontext.cpp
@@ -5,7 +5,7 @@
 // RUN: %env_lsan_opts= %run %t 2>&1
 // RUN: %env_lsan_opts= not %run %t foo 2>&1 | FileCheck %s
 // Missing 'getcontext' and 'makecontext' on Android.
-// UNSUPPORTED: arm,powerpc64,android
+// UNSUPPORTED: arm,aarch64,powerpc64,android
 
 #include "sanitizer_common/sanitizer_ucontext.h"
 #include <stdio.h>