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 | 5b57633a45a5072b4292a98d68eb4ddb4f2b63b6 (patch) | |
| tree | 3c09b2c2ead9c38125e589295068e7a8673deeea /clang/test/Driver/compilation_database.c | |
| parent | 0464c5d958065c571a605f5da3cfd4b729950be7 (diff) | |
| download | bcm5719-llvm-5b57633a45a5072b4292a98d68eb4ddb4f2b63b6.tar.gz bcm5719-llvm-5b57633a45a5072b4292a98d68eb4ddb4f2b63b6.zip | |
[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
llvm-svn: 310950
Diffstat (limited to 'clang/test/Driver/compilation_database.c')
| -rw-r--r-- | clang/test/Driver/compilation_database.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/test/Driver/compilation_database.c b/clang/test/Driver/compilation_database.c index d5bafd41a64..017178d60cd 100644 --- a/clang/test/Driver/compilation_database.c +++ b/clang/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 |
