From 96b7967f83ec2206a38859d908e64d09b32b1292 Mon Sep 17 00:00:00 2001 From: Rafael Espindola Date: Tue, 16 Dec 2014 01:43:41 +0000 Subject: [PATCH] Start adding thin archive support. This is just sufficient for 'ar t' to work. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@224307 91177308-0d34-0410-b5e6-96231b3b80d8 --- include/llvm/Object/Archive.h | 9 ++++----- lib/Object/Archive.cpp | 20 +++++++++++++++++--- test/Object/Inputs/thin.a | Bin 0 -> 474 bytes test/Object/archive-toc.test | 8 ++++++++ 4 files changed, 29 insertions(+), 8 deletions(-) create mode 100644 test/Object/Inputs/thin.a diff --git a/include/llvm/Object/Archive.h b/include/llvm/Object/Archive.h index 7c03dcda172..da6a5419aa0 100644 --- a/include/llvm/Object/Archive.h +++ b/include/llvm/Object/Archive.h @@ -84,7 +84,7 @@ public: return getHeader()->getAccessMode(); } /// \return the size of the archive member without the header or padding. - uint64_t getSize() const { return Data.size() - StartOfFile; } + uint64_t getSize() const; StringRef getBuffer() const { return StringRef(Data.data() + StartOfFile, getSize()); @@ -173,9 +173,7 @@ public: K_COFF }; - Kind kind() const { - return Format; - } + Kind kind() const { return (Kind)Format; } child_iterator child_begin(bool SkipInternal = true) const; child_iterator child_end() const; @@ -201,7 +199,8 @@ private: child_iterator SymbolTable; child_iterator StringTable; child_iterator FirstRegular; - Kind Format; + unsigned Format : 2; + unsigned IsThin : 1; }; } diff --git a/lib/Object/Archive.cpp b/lib/Object/Archive.cpp index d169dbee46e..5aada91ecc0 100644 --- a/lib/Object/Archive.cpp +++ b/lib/Object/Archive.cpp @@ -22,6 +22,7 @@ using namespace llvm; using namespace object; static const char *const Magic = "!\n"; +static const char *const ThinMagic = "!\n"; void Archive::anchor() { } @@ -86,7 +87,10 @@ Archive::Child::Child(const Archive *Parent, const char *Start) const ArchiveMemberHeader *Header = reinterpret_cast(Start); - Data = StringRef(Start, sizeof(ArchiveMemberHeader) + Header->getSize()); + uint64_t Size = sizeof(ArchiveMemberHeader); + if (!Parent->IsThin || Header->getName() == "/" || Header->getName() == "//") + Size += Header->getSize(); + Data = StringRef(Start, Size); // Setup StartOfFile and PaddingBytes. StartOfFile = sizeof(ArchiveMemberHeader); @@ -100,6 +104,12 @@ Archive::Child::Child(const Archive *Parent, const char *Start) } } +uint64_t Archive::Child::getSize() const { + if (Parent->IsThin) + return getHeader()->getSize(); + return Data.size() - StartOfFile; +} + Archive::Child Archive::Child::getNext() const { size_t SpaceToSkip = Data.size(); // If it's odd, add 1 to make it even. @@ -186,9 +196,13 @@ ErrorOr> Archive::create(MemoryBufferRef Source) { Archive::Archive(MemoryBufferRef Source, std::error_code &ec) : Binary(Binary::ID_Archive, Source), SymbolTable(child_end()) { + StringRef Buffer = Data.getBuffer(); // Check for sufficient magic. - if (Data.getBufferSize() < 8 || - StringRef(Data.getBufferStart(), 8) != Magic) { + if (Buffer.startswith(ThinMagic)) { + IsThin = true; + } else if (Buffer.startswith(Magic)) { + IsThin = false; + } else { ec = object_error::invalid_file_type; return; } diff --git a/test/Object/Inputs/thin.a b/test/Object/Inputs/thin.a new file mode 100644 index 0000000000000000000000000000000000000000..bb9d532a7879c7e11339b21e3bdb77ca1fe4e4ab GIT binary patch literal 474 zcmah_OAdlC5Unof6!iiYuoO&;i3=BQIDnfXP}HQBL=6d-@vIKz6Ce>^(l>3VZ{|%` zyZ7yG_Q2>EE_Xw*MGh!+Y--4qa?!nyfe^AJWbvt&JV#vX^5LIe(IGnKWO9)aHf_}v z`VmFb7_%{`QW>obgCfgpEY-7;x-eInRBx3QnUeWkD`_qdwN6q7*2cPE0IaXmG-@Z@ zggBw=IGF8Oa87C1`-y>V`-9#5(_+aMsI9+nfEjJz`{;rT