[llvm] r353873 - [llvm-dwp] Avoid writing the output dwp file when there is an error
Jordan Rupprecht via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 12 12:00:52 PST 2019
Author: rupprecht
Date: Tue Feb 12 12:00:51 2019
New Revision: 353873
URL: http://llvm.org/viewvc/llvm-project?rev=353873&view=rev
Log:
[llvm-dwp] Avoid writing the output dwp file when there is an error
Summary: Use ToolOutputFile to clean up the output file unless dwp actually finishes successfully.
Reviewers: dblaikie
Reviewed By: dblaikie
Subscribers: jdoerfert, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D58130
Modified:
llvm/trunk/test/tools/llvm-dwp/X86/missing_dwo_id.test
llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp
Modified: llvm/trunk/test/tools/llvm-dwp/X86/missing_dwo_id.test
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-dwp/X86/missing_dwo_id.test?rev=353873&r1=353872&r2=353873&view=diff
==============================================================================
--- llvm/trunk/test/tools/llvm-dwp/X86/missing_dwo_id.test (original)
+++ llvm/trunk/test/tools/llvm-dwp/X86/missing_dwo_id.test Tue Feb 12 12:00:51 2019
@@ -1,3 +1,7 @@
+RUN: rm -f %t
RUN: not llvm-dwp %p/../Inputs/missing_dwo_id.dwo -o %t 2>&1 | FileCheck %s
+Make sure we did not leave behind a temporary file:
+RUN: not ls %t
+
CHECK: error: {{.*}}missing_dwo_id.dwo': compile unit missing dwo_id
Modified: llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp?rev=353873&r1=353872&r2=353873&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp (original)
+++ llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp Tue Feb 12 12:00:51 2019
@@ -39,6 +39,7 @@
#include "llvm/Support/Path.h"
#include "llvm/Support/TargetRegistry.h"
#include "llvm/Support/TargetSelect.h"
+#include "llvm/Support/ToolOutputFile.h"
#include "llvm/Support/raw_ostream.h"
using namespace llvm;
@@ -700,23 +701,23 @@ int main(int argc, char **argv) {
// Create the output file.
std::error_code EC;
- raw_fd_ostream OutFile(OutputFilename, EC, sys::fs::F_None);
+ ToolOutputFile OutFile(OutputFilename, EC, sys::fs::F_None);
Optional<buffer_ostream> BOS;
raw_pwrite_stream *OS;
if (EC)
return error(Twine(OutputFilename) + ": " + EC.message(), Context);
- if (OutFile.supportsSeeking()) {
- OS = &OutFile;
+ if (OutFile.os().supportsSeeking()) {
+ OS = &OutFile.os();
} else {
- BOS.emplace(OutFile);
+ BOS.emplace(OutFile.os());
OS = BOS.getPointer();
}
MCTargetOptions MCOptions = InitMCTargetOptionsFromFlags();
std::unique_ptr<MCStreamer> MS(TheTarget->createMCObjectStreamer(
TheTriple, MC, std::unique_ptr<MCAsmBackend>(MAB),
- MAB->createObjectWriter(*OS), std::unique_ptr<MCCodeEmitter>(MCE),
- *MSTI, MCOptions.MCRelaxAll, MCOptions.MCIncrementalLinkerCompatible,
+ MAB->createObjectWriter(*OS), std::unique_ptr<MCCodeEmitter>(MCE), *MSTI,
+ MCOptions.MCRelaxAll, MCOptions.MCIncrementalLinkerCompatible,
/*DWARFMustBeAtTheEnd*/ false));
if (!MS)
return error("no object streamer for target " + TripleName, Context);
@@ -739,4 +740,6 @@ int main(int argc, char **argv) {
}
MS->Finish();
+ OutFile.keep();
+ return 0;
}
More information about the llvm-commits
mailing list