[PATCH] D36437: [clang] Get rid of "%T" expansions

2017-08-15 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL310950: [clang] Get rid of "%T" expansions (authored by kuba.brecka). Changed prior to commit: https://reviews.llvm.org/D36437?vs=110113=111232#toc Repository: rL LLVM

[PATCH] D36437: [clang] Get rid of "%T" expansions

2017-08-08 Thread Reid Kleckner via Phabricator via cfe-commits
rnk accepted this revision. rnk added a comment. Thanks, most of these looks like potential races between tests waiting to happen. Repository: rL LLVM https://reviews.llvm.org/D36437 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D36437: [clang] Get rid of "%T" expansions

2017-08-07 Thread John McCall via Phabricator via cfe-commits
rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. Seems reasonable to me. Repository: rL LLVM https://reviews.llvm.org/D36437 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D36437: [clang] Get rid of "%T" expansions

2017-08-07 Thread Kuba (Brecka) Mracek via Phabricator via cfe-commits
kubamracek created this revision. kubamracek added a project: clang. Herald added subscribers: mehdi_amini, klimek. 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