From 58cb1973de6538d9e5f549404b6ca8b08483db4c Mon Sep 17 00:00:00 2001 From: Alexander Shaposhnikov Date: Thu, 7 Jun 2018 19:41:42 +0000 Subject: [llvm-objcopy] Remove unused field from Object The class Object contains std::shared_ptr OwnedData which is not used anywhere. Besides avoiding two stage initialization the motivation to remove it comes from the plan to add (currently missing) support for static libraries. NFC. Test plan: make check-all Differential revision: https://reviews.llvm.org/D47855 llvm-svn: 334217 --- llvm/tools/llvm-objcopy/Object.cpp | 2 +- llvm/tools/llvm-objcopy/Object.h | 5 ----- 2 files changed, 1 insertion(+), 6 deletions(-) (limited to 'llvm/tools/llvm-objcopy') diff --git a/llvm/tools/llvm-objcopy/Object.cpp b/llvm/tools/llvm-objcopy/Object.cpp index 606e0321288..68420670d52 100644 --- a/llvm/tools/llvm-objcopy/Object.cpp +++ b/llvm/tools/llvm-objcopy/Object.cpp @@ -835,7 +835,7 @@ ElfType ELFReader::getElfType() const { } std::unique_ptr ELFReader::create() const { - auto Obj = llvm::make_unique(Data); + auto Obj = llvm::make_unique(); if (auto *o = dyn_cast>(Bin.get())) { ELFBuilder Builder(*o, *Obj); Builder.build(); diff --git a/llvm/tools/llvm-objcopy/Object.h b/llvm/tools/llvm-objcopy/Object.h index c7939b4a498..48de0e543a0 100644 --- a/llvm/tools/llvm-objcopy/Object.h +++ b/llvm/tools/llvm-objcopy/Object.h @@ -584,7 +584,6 @@ private: using SecPtr = std::unique_ptr; using SegPtr = std::unique_ptr; - std::shared_ptr OwnedData; std::vector Sections; std::vector Segments; @@ -617,10 +616,6 @@ public: StringTableSection *SectionNames = nullptr; SymbolTableSection *SymbolTable = nullptr; - explicit Object(std::shared_ptr Data) - : OwnedData(std::move(Data)) {} - virtual ~Object() = default; - void sortSections(); SectionTableRef sections() { return SectionTableRef(Sections); } ConstRange sections() const { -- cgit v1.2.3