[llvm] r326710 - Fuzzer: remove temporary files after we're done with them.

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 12 07:03:07 PDT 2018


> > is fundamentally broken here and shouldn’t be coddled.

> On Windows, a file can't be deleted if something still holds a reference to it. This is probably somehow related to that.

That would be my guess too, though I stand by my assertion.

> Cleaning up after tests seems like something lit should do; it feels inefficient if we need to do this manually for every test.

That does seem like the natural place to put it. I suppose we could make failure a warning at that level to sidestep the immediate issue. I’ll look into implementing that in case there’s no other way to fix things.

Cheers.

Tim.



More information about the llvm-commits mailing list