[PATCH] D24176: Remove temoprary files.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 17:45:24 PDT 2016


ruiu created this revision.
ruiu added a reviewer: rnk.
ruiu added a subscriber: llvm-commits.

Previously, we created temporary files using llvm::sys::fs::createTemporaryFile
and remove them using llvm::FileRemover. This is error-prone as it is easy to
forget creating FileRemover instances after creating temporary files.
There is actually a temporary file leak bug.

This patch introduces a new class, TemporaryFile, to manage temporary files
in the RTTI style.

https://reviews.llvm.org/D24176

Files:
  COFF/DriverUtils.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24176.70116.patch
Type: text/x-patch
Size: 4890 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160902/0efdf929/attachment.bin>


More information about the llvm-commits mailing list