[llvm] r341515 - Revert rL341509 to fix massive failures on buildbots
Max Kazantsev via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 5 21:40:49 PDT 2018
Author: mkazantsev
Date: Wed Sep 5 21:40:49 2018
New Revision: 341515
URL: http://llvm.org/viewvc/llvm-project?rev=341515&view=rev
Log:
Revert rL341509 to fix massive failures on buildbots
Modified:
llvm/trunk/test/tools/llvm-dwp/X86/merge.test
llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp
Modified: llvm/trunk/test/tools/llvm-dwp/X86/merge.test
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/tools/llvm-dwp/X86/merge.test?rev=341515&r1=341514&r2=341515&view=diff
==============================================================================
--- llvm/trunk/test/tools/llvm-dwp/X86/merge.test (original)
+++ llvm/trunk/test/tools/llvm-dwp/X86/merge.test Wed Sep 5 21:40:49 2018
@@ -1,5 +1,7 @@
-RUN: llvm-dwp %p/../Inputs/merge/notypes/c.dwo %p/../Inputs/merge/notypes/ab.dwp -o - | \
-RUN: llvm-dwarfdump -v - | FileCheck --check-prefix=CHECK --check-prefix=NOTYP %s
+RUN: llvm-dwp %p/../Inputs/merge/notypes/c.dwo %p/../Inputs/merge/notypes/ab.dwp -o %t
+RUN: llvm-dwarfdump -v %t | FileCheck --check-prefix=CHECK --check-prefix=NOTYP %s
+
+FIXME: For some reason, piping straight from llvm-dwp to llvm-dwarfdump -v doesn't behave well - looks like dwarfdump is reading/closes before dwp has finished.
DWP from a DWO (c.dwo) and a DWP (ab.dwp, created from a.dwo and b.dwo)
Make sure the entries for A and B are updated correctly when read/processed from ab.dwp
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=341515&r1=341514&r2=341515&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp (original)
+++ llvm/trunk/tools/llvm-dwp/llvm-dwp.cpp Wed Sep 5 21:40:49 2018
@@ -697,21 +697,13 @@ int main(int argc, char **argv) {
// Create the output file.
std::error_code EC;
raw_fd_ostream 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;
- } else {
- BOS.emplace(OutFile);
- 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),
+ MAB->createObjectWriter(OutFile), std::unique_ptr<MCCodeEmitter>(MCE),
*MSTI, MCOptions.MCRelaxAll, MCOptions.MCIncrementalLinkerCompatible,
/*DWARFMustBeAtTheEnd*/ false));
if (!MS)
More information about the llvm-commits
mailing list