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/Driver/compilation_database.c | |
| 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/Driver/compilation_database.c')
| -rw-r--r-- | test/Driver/compilation_database.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/Driver/compilation_database.c b/test/Driver/compilation_database.c index d5bafd41a6..017178d60c 100644 --- a/test/Driver/compilation_database.c +++ b/test/Driver/compilation_database.c @@ -1,4 +1,4 @@ -// RUN: cd "%T" +// RUN: mkdir -p %t && cd %t // RUN: %clang -MD -MP --sysroot=somewhere -c -x c %s -xc++ %s -Wall -MJ - -no-canonical-prefixes 2>&1 | FileCheck %s // RUN: not %clang -c -x c %s -MJ %s/non-existant -no-canonical-prefixes 2>&1 | FileCheck --check-prefix=ERROR %s |