[PATCH] D67424: [Support] Add overload writeFileAtomically(std::function Writer)

[Jan Korous via Phabricator via llvm-commits]

jkorous marked 3 inline comments as done.
jkorous added inline comments.


================
Comment at: llvm/include/llvm/Support/FileUtilities.h:107
+  /// \returns \c AtomicFileWriteError in case of error.
   llvm::Error writeFileAtomically(StringRef TempPathModel, StringRef FinalPath,
                                   StringRef Buffer);
----------------
JDevlieghere wrote:
> I wonder if the temp path should be configurable or at least make it a default argument. Removing that would definitely make the call sites a little simpler. At least for LLDB I don't think the temp path matters.
Of course, good point! I'll create a separate patch.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67424/new/

https://reviews.llvm.org/D67424