[llvm-lit] remove %t files before running a test

James Molloy james at jamesmolloy.co.uk
Thu May 14 05:27:22 PDT 2015


Hi Tzafrir,

This sounds like a good idea to me.

+    subprocess.call(['rm','-rf',path+"*"])



This is super dangerous. Why the -f ?

Cheers,

James

On Thu, 14 May 2015 at 13:22 Tzafrir Poupko <tzafrir11 at gmail.com> wrote:

> Before running a test, purge all %t files used by previous tests.
> Remove any custom file used in test which was prefixed by %t.
> There are over 300 clang tests that run "rm -rf %t" as first run command,
> this patch will make it automaticlly.
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150514/9e5cc3d9/attachment.html>


More information about the llvm-commits mailing list