]> granicus.if.org Git - clang/commit
[clang] Get rid of "%T" expansions
authorKuba Mracek <mracek@apple.com>
Tue, 15 Aug 2017 19:47:06 +0000 (19:47 +0000)
committerKuba Mracek <mracek@apple.com>
Tue, 15 Aug 2017 19:47:06 +0000 (19:47 +0000)
commitd9232a8d2393a568d1e6732eefe09996df5f3cbd
tree075c6f329229809ce7ee4b1fa2de829283775ab0
parent709ce01f5878ac12902d0b31d363ccd5a31fae74
[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
23 files changed:
test/Analysis/html-diags.c
test/CoverageMapping/abspath.cpp
test/Driver/compilation_database.c
test/Driver/cpath.c
test/Driver/darwin-ld-lto.c
test/Driver/linker-opts.c
test/Driver/output-file-cleanup.c
test/Driver/parse-progname.c
test/Driver/ps4-linker-non-win.c
test/Driver/ps4-linker-win.c
test/Driver/warning-options.cpp
test/FixIt/fixit-include.c
test/Format/style-on-command-line.cpp
test/Lexer/case-insensitive-include-ms.c
test/Lexer/case-insensitive-include-pr31836.sh
test/Lexer/case-insensitive-include.c
test/Lexer/case-insensitive-system-include.c
test/Modules/crash-typo-correction-visibility.cpp
test/Modules/modules-cache-path-canonicalization.m
test/PCH/case-insensitive-include.c
test/PCH/include-timestamp.cpp
test/Preprocessor/cuda-types.cu
test/Tooling/clang-diff-basic.cpp