[PATCH] D35396: [lit] Make %T return a per-test temporary directory

Kuba (Brecka) Mracek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 26 17:00:28 PDT 2017


kubamracek added a comment.

In https://reviews.llvm.org/D35396#822242, @rnk wrote:

> In https://reviews.llvm.org/D35396#809760, @kubamracek wrote:
>
> > > lit already ensures that tmpDir exists, so that's what this change will do. Look for callers of getTempPaths.
> >
> > Oh.  That's not what I meant then.  I'll update the patch.
>
>
> This doesn't seem like it was addressed? As is, surely this will regress lit startup time.


This was addressed, I changed the patch.  `dirname(tmpBase)` is later being mkdir'd, but `tmpDir` isn't.


https://reviews.llvm.org/D35396





More information about the llvm-commits mailing list