[compiler-rt] d9b062a - [TSan][Darwin] Remove unnecessary lit substitution

Julian Lettner via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 20 13:02:42 PDT 2020


Author: Julian Lettner
Date: 2020-08-20T13:00:32-07:00
New Revision: d9b062ad8777c283f9a1099a3ce29248348e5f40

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

LOG: [TSan][Darwin] Remove unnecessary lit substitution

We don't test on very old versions of Apple platforms anymore.  The
following lit substitution concerning the minimum deployment target for
ARC support can be removed.

```
%darwin_min_target_with_full_runtime_arc_support -> 10.11
```

Differential Revision: https://reviews.llvm.org/D85803

Added: 
    

Modified: 
    compiler-rt/test/lit.common.cfg.py
    compiler-rt/test/tsan/Darwin/norace-objcxx-run-time.mm
    compiler-rt/test/tsan/Darwin/objc-synchronize-cycle-tagged.mm
    compiler-rt/test/tsan/Darwin/objc-synchronize-cycle.mm
    compiler-rt/test/tsan/Darwin/objc-synchronize-nested-recursive.mm
    compiler-rt/test/tsan/Darwin/objc-synchronize-tagged.mm
    compiler-rt/test/tsan/Darwin/objc-synchronize.mm

Removed: 
    


################################################################################
diff  --git a/compiler-rt/test/lit.common.cfg.py b/compiler-rt/test/lit.common.cfg.py
index d773f3766fd3..53b0a47f11a9 100644
--- a/compiler-rt/test/lit.common.cfg.py
+++ b/compiler-rt/test/lit.common.cfg.py
@@ -268,9 +268,7 @@
 # to the macOS version that first contained the relevant feature.
 darwin_min_deployment_target_substitutions = {
   '%macos_min_target_10_11': '10.11',
-  # rdar://problem/22207160
-  '%darwin_min_target_with_full_runtime_arc_support': '10.11',
-  '%darwin_min_target_with_tls_support': '10.12',
+  '%darwin_min_target_with_tls_support': '10.12',  # TLS requires watchOS 3+ simulator
 }
 
 if config.host_os == 'Darwin':

diff  --git a/compiler-rt/test/tsan/Darwin/norace-objcxx-run-time.mm b/compiler-rt/test/tsan/Darwin/norace-objcxx-run-time.mm
index b6a38d702240..0cf729e7f2d8 100644
--- a/compiler-rt/test/tsan/Darwin/norace-objcxx-run-time.mm
+++ b/compiler-rt/test/tsan/Darwin/norace-objcxx-run-time.mm
@@ -1,4 +1,4 @@
-// RUN: %clang_tsan %s -lc++ -fobjc-arc -lobjc -o %t -framework Foundation %darwin_min_target_with_full_runtime_arc_support
+// RUN: %clang_tsan %s -lc++ -fobjc-arc -lobjc -o %t -framework Foundation
 // RUN: %env_tsan_opts=ignore_interceptors_accesses=1 %run %t 2>&1 | FileCheck %s
 
 // Check that we do not report races between:

diff  --git a/compiler-rt/test/tsan/Darwin/objc-synchronize-cycle-tagged.mm b/compiler-rt/test/tsan/Darwin/objc-synchronize-cycle-tagged.mm
index 5806e8af925e..c438984f1e5d 100644
--- a/compiler-rt/test/tsan/Darwin/objc-synchronize-cycle-tagged.mm
+++ b/compiler-rt/test/tsan/Darwin/objc-synchronize-cycle-tagged.mm
@@ -1,4 +1,4 @@
-// RUN: %clangxx_tsan %s -o %t -framework Foundation -fobjc-arc %darwin_min_target_with_full_runtime_arc_support
+// RUN: %clangxx_tsan %s -o %t -framework Foundation -fobjc-arc
 // RUN:     %run %t 6 2>&1 | FileCheck %s --check-prefix=SIX
 // RUN: not %run %t 7 2>&1 | FileCheck %s --check-prefix=SEVEN
 

diff  --git a/compiler-rt/test/tsan/Darwin/objc-synchronize-cycle.mm b/compiler-rt/test/tsan/Darwin/objc-synchronize-cycle.mm
index fb816311580b..006716fefcd3 100644
--- a/compiler-rt/test/tsan/Darwin/objc-synchronize-cycle.mm
+++ b/compiler-rt/test/tsan/Darwin/objc-synchronize-cycle.mm
@@ -1,4 +1,4 @@
-// RUN: %clangxx_tsan %s -o %t -framework Foundation -fobjc-arc %darwin_min_target_with_full_runtime_arc_support
+// RUN: %clangxx_tsan %s -o %t -framework Foundation -fobjc-arc
 // RUN:                                   not %run %t 2>&1 | FileCheck %s
 // RUN: %env_tsan_opts=detect_deadlocks=1 not %run %t 2>&1 | FileCheck %s
 // RUN: %env_tsan_opts=detect_deadlocks=0     %run %t 2>&1 | FileCheck %s --check-prefix=DISABLED

diff  --git a/compiler-rt/test/tsan/Darwin/objc-synchronize-nested-recursive.mm b/compiler-rt/test/tsan/Darwin/objc-synchronize-nested-recursive.mm
index ab6643e5c54e..be48a0ed59ad 100644
--- a/compiler-rt/test/tsan/Darwin/objc-synchronize-nested-recursive.mm
+++ b/compiler-rt/test/tsan/Darwin/objc-synchronize-nested-recursive.mm
@@ -1,4 +1,4 @@
-// RUN: %clangxx_tsan %s -o %t -framework Foundation -fobjc-arc %darwin_min_target_with_full_runtime_arc_support
+// RUN: %clangxx_tsan %s -o %t -framework Foundation -fobjc-arc
 // RUN: %run %t 2>&1 | FileCheck %s
 
 #import <Foundation/Foundation.h>

diff  --git a/compiler-rt/test/tsan/Darwin/objc-synchronize-tagged.mm b/compiler-rt/test/tsan/Darwin/objc-synchronize-tagged.mm
index ab0af46ec4af..2b2c08b8c606 100644
--- a/compiler-rt/test/tsan/Darwin/objc-synchronize-tagged.mm
+++ b/compiler-rt/test/tsan/Darwin/objc-synchronize-tagged.mm
@@ -1,4 +1,4 @@
-// RUN: %clangxx_tsan %s -o %t -framework Foundation -fobjc-arc %darwin_min_target_with_full_runtime_arc_support
+// RUN: %clangxx_tsan %s -o %t -framework Foundation -fobjc-arc
 // RUN: %run %t 2>&1 | FileCheck %s
 
 #import <Foundation/Foundation.h>

diff  --git a/compiler-rt/test/tsan/Darwin/objc-synchronize.mm b/compiler-rt/test/tsan/Darwin/objc-synchronize.mm
index 0bf06370a9b8..85aaa24d42fd 100644
--- a/compiler-rt/test/tsan/Darwin/objc-synchronize.mm
+++ b/compiler-rt/test/tsan/Darwin/objc-synchronize.mm
@@ -1,4 +1,4 @@
-// RUN: %clangxx_tsan %s -o %t -framework Foundation -fobjc-arc %darwin_min_target_with_full_runtime_arc_support
+// RUN: %clangxx_tsan %s -o %t -framework Foundation -fobjc-arc
 // RUN: %run %t 2>&1 | FileCheck %s
 
 #import <Foundation/Foundation.h>


        


More information about the llvm-commits mailing list