[PATCH] D48189: [llvm] Document "%T" as deprecated in TestingGuide.rst

[Peter Wu via Phabricator via llvm-commits]

Lekensteyn added a comment.

One small comment, LGTM otherwise.



================
Comment at: docs/TestingGuide.rst:464
+   Directory of ``%t``. Deprecated. Shouldn't be used, because it can be easily
+   misused and cause race conditions between tests. 
 
----------------
What about suggesting an alternative like:

    Use ``rm -rf %t && mkdir %t`` instead if a temporary directory is necessary.

Perhaps that could even replace the `Shouldn't be used, because [reason]` sentence if brevity is desired.


https://reviews.llvm.org/D48189