diff options
| author | Kuba Mracek <mracek@apple.com> | 2017-08-15 20:29:24 +0000 |
|---|---|---|
| committer | Kuba Mracek <mracek@apple.com> | 2017-08-15 20:29:24 +0000 |
| commit | 17ee427ef35d7521f90f28a7db83fddb4aadd475 (patch) | |
| tree | 0d63ef2fcb684b5fbddf5e45cca3acfded4af8b6 /llvm/test/tools/llvm-strings/nested-archives.test | |
| parent | cea6267cc6114bc3ee89e66ede36239841d813c0 (diff) | |
| download | bcm5719-llvm-17ee427ef35d7521f90f28a7db83fddb4aadd475.tar.gz bcm5719-llvm-17ee427ef35d7521f90f28a7db83fddb4aadd475.zip | |
[llvm] Get rid of "%T" expansions
The %T lit expansion expands to a common directory shared between all the tests in the same directory, which is unexpected and unintuitive, and more importantly, it's been a source of subtle race conditions and flaky tests. In https://reviews.llvm.org/D35396, it was agreed that it would be best to simply ban %T and only keep %t, which is unique to each test. When a test needs a temporary directory, it can just create one using mkdir %t.
This patch removes %T in llvm.
Differential Revision: https://reviews.llvm.org/D36495
llvm-svn: 310953
Diffstat (limited to 'llvm/test/tools/llvm-strings/nested-archives.test')
| -rw-r--r-- | llvm/test/tools/llvm-strings/nested-archives.test | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/llvm/test/tools/llvm-strings/nested-archives.test b/llvm/test/tools/llvm-strings/nested-archives.test index 09ad0043c6b..a0a395c8d9b 100644 --- a/llvm/test/tools/llvm-strings/nested-archives.test +++ b/llvm/test/tools/llvm-strings/nested-archives.test @@ -1,8 +1,7 @@ -RUN: rm -f %T/inner.ar -RUN: llvm-ar -format gnu crs %T/inner.a %S/Inputs/abcd -RUN: rm -f %T/outer.ar -RUN: llvm-ar -format gnu crs %T/outer.a %T/inner.a -RUN: llvm-strings %T/outer.a | FileCheck %s +RUN: rm -rf %t && mkdir -p %t +RUN: llvm-ar -format gnu crs %t/inner.a %S/Inputs/abcd +RUN: llvm-ar -format gnu crs %t/outer.a %t/inner.a +RUN: llvm-strings %t/outer.a | FileCheck %s CHECK: !<arch> CHECK: inner.a/ 0 0 0 644 72 ` |
