[PATCH] D49860: [dsymutil] Simplify temporary file handling.

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 26 08:35:02 PDT 2018


JDevlieghere created this revision.
JDevlieghere added reviewers: jmorse, friss, probinson.
Herald added a subscriber: hiraditya.

Dsymutil's update functionality was broken on Windows because we tried to rename a file while we're holding open handles to that file. TempFile provides a solution for this through its `keep(Twine)` method. This patch changes dsymutil to make use of that functionality.


Repository:
  rL LLVM

https://reviews.llvm.org/D49860

Files:
  llvm/lib/Support/Path.cpp
  llvm/test/tools/dsymutil/X86/accelerator.test
  llvm/test/tools/dsymutil/X86/update-one-CU.test
  llvm/test/tools/dsymutil/X86/update.test
  llvm/tools/dsymutil/MachOUtils.cpp
  llvm/tools/dsymutil/MachOUtils.h
  llvm/tools/dsymutil/dsymutil.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49860.157495.patch
Type: text/x-patch
Size: 8690 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180726/f5ff6c96/attachment.bin>


More information about the llvm-commits mailing list