diff options
author | Kuba Mracek <mracek@apple.com> | 2017-08-15 19:47:06 +0000 |
---|---|---|
committer | Kuba Mracek <mracek@apple.com> | 2017-08-15 19:47:06 +0000 |
commit | d9232a8d2393a568d1e6732eefe09996df5f3cbd (patch) | |
tree | 075c6f329229809ce7ee4b1fa2de829283775ab0 /test/CoverageMapping | |
parent | 709ce01f5878ac12902d0b31d363ccd5a31fae74 (diff) | |
download | clang-d9232a8d2393a568d1e6732eefe09996df5f3cbd.tar.gz |
[clang] 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 clang.
Differential Revision: https://reviews.llvm.org/D36437
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@310950 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/CoverageMapping')
-rw-r--r-- | test/CoverageMapping/abspath.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/CoverageMapping/abspath.cpp b/test/CoverageMapping/abspath.cpp index 667172e32c..4b92a3773d 100644 --- a/test/CoverageMapping/abspath.cpp +++ b/test/CoverageMapping/abspath.cpp @@ -4,7 +4,7 @@ // RMDOTS-NOT: Inputs // RMDOTS: " -// RUN: cd %T && mkdir -p test && cd test +// RUN: mkdir -p %t/test && cd %t/test // RUN: echo "void f1() {}" > f1.c // RUN: %clang_cc1 -fprofile-instrument=clang -fcoverage-mapping -emit-llvm -main-file-name abspath.cpp ../test/f1.c -o - | FileCheck -check-prefix=RELPATH %s |