[PATCH] D154329: [lldb] Replace llvm::writeFileAtomically with llvm::writeToOutput API.
Alexey Lapshin via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 3 05:41:23 PDT 2023
avl added inline comments.
================
Comment at: lldb/tools/lldb-server/lldb-platform.cpp:112
return Status("Failed to atomically write file %s",
file_spec.GetPath().c_str());
return status;
----------------
probably, it would be better to add error text here?
```
return Status("Failed to atomically write file %s: %s",
file_spec.GetPath().c_str(), toString(std::move(Err)).c_str());
```
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154329/new/
https://reviews.llvm.org/D154329
More information about the cfe-commits
mailing list