From 578f2cdd2d349723f5047237e6cea39285a54bde Mon Sep 17 00:00:00 2001 From: David Blaikie Date: Tue, 1 Mar 2016 20:40:02 +0000 Subject: [PATCH] llvm-dwp: Keep ObjectFiles alive until object emission their contents can be referenced directly rather than copied llvm-svn: 262389 --- llvm/tools/llvm-dwp/llvm-dwp.cpp | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/llvm/tools/llvm-dwp/llvm-dwp.cpp b/llvm/tools/llvm-dwp/llvm-dwp.cpp index ceadfb1..131d5b3 100644 --- a/llvm/tools/llvm-dwp/llvm-dwp.cpp +++ b/llvm/tools/llvm-dwp/llvm-dwp.cpp @@ -358,11 +358,17 @@ static std::error_code write(MCStreamer &Out, ArrayRef Inputs) { uint32_t ContributionOffsets[8] = {}; + SmallVector, 128> Objects; + Objects.reserve(Inputs.size()); for (const auto &Input : Inputs) { auto ErrOrObj = object::ObjectFile::createObjectFile(Input); if (!ErrOrObj) return ErrOrObj.getError(); + Objects.push_back(std::move(*ErrOrObj)); + + auto &Obj = *Objects.back().getBinary(); + UnitIndexEntry CurEntry = {}; StringRef CurStrSection; @@ -375,7 +381,7 @@ static std::error_code write(MCStreamer &Out, ArrayRef Inputs) { SmallVector, 4> UncompressedSections; - for (const auto &Section : ErrOrObj->getBinary()->sections()) { + for (const auto &Section : Obj.sections()) { if (Section.isBSS()) continue; if (Section.isVirtual()) @@ -452,8 +458,7 @@ static std::error_code write(MCStreamer &Out, ArrayRef Inputs) { if (!CurCUIndexSection.empty()) { DWARFUnitIndex CUIndex(DW_SECT_INFO); - DataExtractor CUIndexData(CurCUIndexSection, - ErrOrObj->getBinary()->isLittleEndian(), 0); + DataExtractor CUIndexData(CurCUIndexSection, Obj.isLittleEndian(), 0); if (!CUIndex.parse(CUIndexData)) return make_error_code(std::errc::invalid_argument); @@ -491,8 +496,7 @@ static std::error_code write(MCStreamer &Out, ArrayRef Inputs) { if (CurTUIndexSection.empty()) return make_error_code(std::errc::invalid_argument); DWARFUnitIndex TUIndex(DW_SECT_TYPES); - DataExtractor TUIndexData(CurTUIndexSection, - ErrOrObj->getBinary()->isLittleEndian(), 0); + DataExtractor TUIndexData(CurTUIndexSection, Obj.isLittleEndian(), 0); if (!TUIndex.parse(TUIndexData)) return make_error_code(std::errc::invalid_argument); addAllTypesFromDWP(Out, TypeIndexEntries, TUIndex, TypesSection, @@ -537,7 +541,7 @@ static std::error_code write(MCStreamer &Out, ArrayRef Inputs) { writeIndex(Out, MCOFI.getDwarfCUIndexSection(), ContributionOffsets, IndexEntries); - + Out.Finish(); return std::error_code(); } @@ -608,6 +612,4 @@ int main(int argc, char **argv) { if (auto Err = write(*MS, InputFiles)) return error(Err.message(), "Writing DWP file"); - - MS->Finish(); } -- 2.7.4