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
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
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
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