[llvm] r303490 - COFF: migrate def parser from LLD to LLVM [1/2]

Galina Kistanova via llvm-commits llvm-commits at lists.llvm.org
Wed May 31 11:23:40 PDT 2017


Hello Martell,

This commit added some warnings, please take care of these?

Thanks

Galina


/home/buildslave/am1i-slv2/builddir/llvm/lib/Object/COFFImportFile.cpp: In
member function ‘llvm::NewArchiveMember
llvm::object::{anonymous}::ObjectFactory::createImportDescriptor(std::vector<unsigned
char, std::allocator<unsigned char> >&)’:
/home/buildslave/am1i-slv2/builddir/llvm/lib/Object/COFFImportFile.cpp:288:60:
warning: dereferencing type-punned pointer will break strict-aliasing rules
[-Wstrict-aliasing]
   reinterpret_cast<StringTableOffset &>(SymbolTable[0].Name).Offset =
                                                            ^
/home/buildslave/am1i-slv2/builddir/llvm/lib/Object/COFFImportFile.cpp:290:60:
warning: dereferencing type-punned pointer will break strict-aliasing rules
[-Wstrict-aliasing]
   reinterpret_cast<StringTableOffset &>(SymbolTable[5].Name).Offset =
                                                            ^
/home/buildslave/am1i-slv2/builddir/llvm/lib/Object/COFFImportFile.cpp:292:60:
warning: dereferencing type-punned pointer will break strict-aliasing rules
[-Wstrict-aliasing]
   reinterpret_cast<StringTableOffset &>(SymbolTable[6].Name).Offset =
                                                            ^
/home/buildslave/am1i-slv2/builddir/llvm/lib/Object/COFFImportFile.cpp: In
member function ‘llvm::NewArchiveMember
llvm::object::{anonymous}::ObjectFactory::createNullImportDescriptor(std::vector<unsigned
char, std::allocator<unsigned char> >&)’:
/home/buildslave/am1i-slv2/builddir/llvm/lib/Object/COFFImportFile.cpp:357:60:
warning: dereferencing type-punned pointer will break strict-aliasing rules
[-Wstrict-aliasing]
   reinterpret_cast<StringTableOffset &>(SymbolTable[0].Name).Offset =
                                                            ^
/home/buildslave/am1i-slv2/builddir/llvm/lib/Object/COFFImportFile.cpp: In
member function ‘llvm::NewArchiveMember
llvm::object::{anonymous}::ObjectFactory::createNullThunk(std::vector<unsigned
char, std::allocator<unsigned char> >&)’:
/home/buildslave/am1i-slv2/builddir/llvm/lib/Object/COFFImportFile.cpp:440:60:
warning: dereferencing type-punned pointer will break strict-aliasing rules
[-Wstrict-aliasing]
   reinterpret_cast<StringTableOffset &>(SymbolTable[0].Name).Offset =
                                                            ^


On Sat, May 20, 2017 at 12:56 PM, Martell Malone via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: martell
> Date: Sat May 20 14:56:29 2017
> New Revision: 303490
>
> URL: http://llvm.org/viewvc/llvm-project?rev=303490&view=rev
> Log:
> COFF: migrate def parser from LLD to LLVM [1/2]
>
> This is split up into two commits.
> The will create the DEF parser in LLVM.
> Check the following commit to see the removal from LLD
>
> Reviewers: ruiu
>
> Differential Revision: https://reviews.llvm.org/D32689
>
> Added:
>     llvm/trunk/include/llvm/Object/COFFModuleDefinition.h
>     llvm/trunk/lib/Object/COFFImportFile.cpp
>     llvm/trunk/lib/Object/COFFModuleDefinition.cpp
> Modified:
>     llvm/trunk/include/llvm/Object/COFFImportFile.h
>     llvm/trunk/lib/Object/CMakeLists.txt
>
> Modified: llvm/trunk/include/llvm/Object/COFFImportFile.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/
> llvm/Object/COFFImportFile.h?rev=303490&r1=303489&r2=303490&view=diff
> ============================================================
> ==================
> --- llvm/trunk/include/llvm/Object/COFFImportFile.h (original)
> +++ llvm/trunk/include/llvm/Object/COFFImportFile.h Sat May 20 14:56:29
> 2017
> @@ -9,13 +9,15 @@
>  //
>  // COFF short import file is a special kind of file which contains
>  // only symbol names for DLL-exported symbols. This class implements
> -// SymbolicFile interface for the file.
> +// exporting of Symbols to create libraries and a SymbolicFile
> +// interface for the file type.
>  //
>  //===-------------------------------------------------------
> ---------------===//
>
>  #ifndef LLVM_OBJECT_COFF_IMPORT_FILE_H
>  #define LLVM_OBJECT_COFF_IMPORT_FILE_H
>
> +#include "llvm/ADT/ArrayRef.h"
>  #include "llvm/Object/COFF.h"
>  #include "llvm/Object/IRObjectFile.h"
>  #include "llvm/Object/ObjectFile.h"
> @@ -68,6 +70,36 @@ private:
>    }
>  };
>
> +struct COFFShortExport {
> +  std::string Name;
> +  std::string ExtName;
> +
> +  uint16_t Ordinal = 0;
> +  bool Noname = false;
> +  bool Data = false;
> +  bool Private = false;
> +  bool Constant = false;
> +
> +  bool isWeak() {
> +    return ExtName.size() && ExtName != Name;
> +  }
> +
> +  friend bool operator==(const COFFShortExport &L, const COFFShortExport
> &R) {
> +    return L.Name == R.Name && L.ExtName == R.ExtName &&
> +            L.Ordinal == R.Ordinal && L.Noname == R.Noname &&
> +            L.Data == R.Data && L.Private == R.Private;
> +  }
> +
> +  friend bool operator!=(const COFFShortExport &L, const COFFShortExport
> &R) {
> +    return !(L == R);
> +  }
> +};
> +
> +std::error_code writeImportLibrary(StringRef DLLName,
> +                                   StringRef Path,
> +                                   ArrayRef<COFFShortExport> Exports,
> +                                   COFF::MachineTypes Machine);
> +
>  } // namespace object
>  } // namespace llvm
>
>
> Added: llvm/trunk/include/llvm/Object/COFFModuleDefinition.h
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Object/
> COFFModuleDefinition.h?rev=303490&view=auto
> ============================================================
> ==================
> --- llvm/trunk/include/llvm/Object/COFFModuleDefinition.h (added)
> +++ llvm/trunk/include/llvm/Object/COFFModuleDefinition.h Sat May 20
> 14:56:29 2017
> @@ -0,0 +1,49 @@
> +//===--- COFFModuleDefinition.h ---------------------------------*- C++
> -*-===//
> +//
> +//                     The LLVM Compiler Infrastructure
> +//
> +// This file is distributed under the University of Illinois Open Source
> +// License. See LICENSE.TXT for details.
> +//
> +//===------------------------------------------------------
> ----------------===//
> +//
> +// Windows-specific.
> +// A parser for the module-definition file (.def file).
> +// Parsed results are directly written to Config global variable.
> +//
> +// The format of module-definition files are described in this document:
> +// https://msdn.microsoft.com/en-us/library/28d6s79h.aspx
> +//
> +//===------------------------------------------------------
> ----------------===//
> +
> +
> +#ifndef LLVM_OBJECT_COFF_MODULE_DEFINITION_H
> +#define LLVM_OBJECT_COFF_MODULE_DEFINITION_H
> +
> +#include "llvm/Object/COFFImportFile.h"
> +#include "llvm/Object/COFF.h"
> +
> +namespace llvm {
> +namespace object {
> +
> +struct COFFModuleDefinition {
> +  std::vector<COFFShortExport> Exports;
> +  std::string OutputFile;
> +  uint64_t ImageBase = 0;
> +  uint64_t StackReserve = 0;
> +  uint64_t StackCommit = 0;
> +  uint64_t HeapReserve = 0;
> +  uint64_t HeapCommit = 0;
> +  uint32_t MajorImageVersion = 0;
> +  uint32_t MinorImageVersion = 0;
> +  uint32_t MajorOSVersion = 0;
> +  uint32_t MinorOSVersion = 0;
> +};
> +
> +Expected<COFFModuleDefinition>
> +parseCOFFModuleDefinition(MemoryBufferRef MB, COFF::MachineTypes
> Machine);
> +
> +} // End namespace object.
> +} // End namespace llvm.
> +
> +#endif
>
> Modified: llvm/trunk/lib/Object/CMakeLists.txt
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Object/
> CMakeLists.txt?rev=303490&r1=303489&r2=303490&view=diff
> ============================================================
> ==================
> --- llvm/trunk/lib/Object/CMakeLists.txt (original)
> +++ llvm/trunk/lib/Object/CMakeLists.txt Sat May 20 14:56:29 2017
> @@ -2,6 +2,8 @@ add_llvm_library(LLVMObject
>    Archive.cpp
>    ArchiveWriter.cpp
>    Binary.cpp
> +  COFFImportFile.cpp
> +  COFFModuleDefinition.cpp
>    COFFObjectFile.cpp
>    Decompressor.cpp
>    ELF.cpp
>
> Added: llvm/trunk/lib/Object/COFFImportFile.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Object/
> COFFImportFile.cpp?rev=303490&view=auto
> ============================================================
> ==================
> --- llvm/trunk/lib/Object/COFFImportFile.cpp (added)
> +++ llvm/trunk/lib/Object/COFFImportFile.cpp Sat May 20 14:56:29 2017
> @@ -0,0 +1,527 @@
> +//===- COFFImportFile.cpp - COFF short import file implementation
> ---------===//
> +//
> +//                     The LLVM Compiler Infrastructure
> +//
> +// This file is distributed under the University of Illinois Open Source
> +// License. See LICENSE.TXT for details.
> +//
> +//===------------------------------------------------------
> ----------------===//
> +//
> +// This file defines the writeImportLibrary function.
> +//
> +//===------------------------------------------------------
> ----------------===//
> +
> +#include "llvm/Object/COFFImportFile.h"
> +#include "llvm/ADT/ArrayRef.h"
> +#include "llvm/Object/Archive.h"
> +#include "llvm/Object/ArchiveWriter.h"
> +#include "llvm/Object/COFF.h"
> +#include "llvm/Support/Error.h"
> +#include "llvm/Support/Path.h"
> +
> +#include <cstdint>
> +#include <map>
> +#include <set>
> +#include <string>
> +#include <vector>
> +
> +using namespace llvm::COFF;
> +using namespace llvm::object;
> +using namespace llvm;
> +
> +namespace llvm {
> +namespace object {
> +
> +static bool is32bit(MachineTypes Machine) {
> +  switch (Machine) {
> +  default:
> +    llvm_unreachable("unsupported machine");
> +  case IMAGE_FILE_MACHINE_AMD64:
> +    return false;
> +  case IMAGE_FILE_MACHINE_ARMNT:
> +  case IMAGE_FILE_MACHINE_I386:
> +    return true;
> +  }
> +}
> +
> +static uint16_t getImgRelRelocation(MachineTypes Machine) {
> +  switch (Machine) {
> +  default:
> +    llvm_unreachable("unsupported machine");
> +  case IMAGE_FILE_MACHINE_AMD64:
> +    return IMAGE_REL_AMD64_ADDR32NB;
> +  case IMAGE_FILE_MACHINE_ARMNT:
> +    return IMAGE_REL_ARM_ADDR32NB;
> +  case IMAGE_FILE_MACHINE_I386:
> +    return IMAGE_REL_I386_DIR32NB;
> +  }
> +}
> +
> +template <class T> static void append(std::vector<uint8_t> &B, const T
> &Data) {
> +  size_t S = B.size();
> +  B.resize(S + sizeof(T));
> +  memcpy(&B[S], &Data, sizeof(T));
> +}
> +
> +static void writeStringTable(std::vector<uint8_t> &B,
> +                             ArrayRef<const std::string> Strings) {
> +  // The COFF string table consists of a 4-byte value which is the size
> of the
> +  // table, including the length field itself.  This value is followed by
> the
> +  // string content itself, which is an array of null-terminated C-style
> +  // strings.  The termination is important as they are referenced to by
> offset
> +  // by the symbol entity in the file format.
> +
> +  size_t Pos = B.size();
> +  size_t Offset = B.size();
> +
> +  // Skip over the length field, we will fill it in later as we will have
> +  // computed the length while emitting the string content itself.
> +  Pos += sizeof(uint32_t);
> +
> +  for (const auto &S : Strings) {
> +    B.resize(Pos + S.length() + 1);
> +    strcpy(reinterpret_cast<char *>(&B[Pos]), S.c_str());
> +    Pos += S.length() + 1;
> +  }
> +
> +  // Backfill the length of the table now that it has been computed.
> +  support::ulittle32_t Length(B.size() - Offset);
> +  support::endian::write32le(&B[Offset], Length);
> +}
> +
> +static ImportNameType getNameType(StringRef Sym, StringRef ExtName,
> +                                  MachineTypes Machine) {
> +  if (Sym != ExtName)
> +    return IMPORT_NAME_UNDECORATE;
> +  if (Machine == IMAGE_FILE_MACHINE_I386 && Sym.startswith("_"))
> +    return IMPORT_NAME_NOPREFIX;
> +  return IMPORT_NAME;
> +}
> +
> +static Expected<std::string> replace(StringRef S, StringRef From,
> +                                     StringRef To) {
> +  size_t Pos = S.find(From);
> +
> +  // From and To may be mangled, but substrings in S may not.
> +  if (Pos == StringRef::npos && From.startswith("_") &&
> To.startswith("_")) {
> +    From = From.substr(1);
> +    To = To.substr(1);
> +    Pos = S.find(From);
> +  }
> +
> +  if (Pos == StringRef::npos) {
> +    return make_error<StringError>(
> +      Twine(S + ": replacing '" + From + "' with '" + To + "' failed")
> +      .getSingleStringRef(), object_error::parse_failed);
> +  }
> +
> +  return (Twine(S.substr(0, Pos)) + To + S.substr(Pos +
> From.size())).str();
> +}
> +
> +static const std::string NullImportDescriptorSymbolName =
> +    "__NULL_IMPORT_DESCRIPTOR";
> +
> +namespace {
> +// This class constructs various small object files necessary to support
> linking
> +// symbols imported from a DLL.  The contents are pretty strictly defined
> and
> +// nearly entirely static.  The details of the structures files are
> defined in
> +// WINNT.h and the PE/COFF specification.
> +class ObjectFactory {
> +  using u16 = support::ulittle16_t;
> +  using u32 = support::ulittle32_t;
> +  MachineTypes Machine;
> +  BumpPtrAllocator Alloc;
> +  StringRef DLLName;
> +  StringRef Library;
> +  std::string ImportDescriptorSymbolName;
> +  std::string NullThunkSymbolName;
> +
> +public:
> +  ObjectFactory(StringRef S, MachineTypes M)
> +      : Machine(M), DLLName(S), Library(S.drop_back(4)),
> +        ImportDescriptorSymbolName(("__IMPORT_DESCRIPTOR_" +
> Library).str()),
> +        NullThunkSymbolName(("\x7f" + Library +
> "_NULL_THUNK_DATA").str()) {}
> +
> +  // Creates an Import Descriptor.  This is a small object file which
> contains a
> +  // reference to the terminators and contains the library name (entry)
> for the
> +  // import name table.  It will force the linker to construct the
> necessary
> +  // structure to import symbols from the DLL.
> +  NewArchiveMember createImportDescriptor(std::vector<uint8_t> &Buffer);
> +
> +  // Creates a NULL import descriptor.  This is a small object file whcih
> +  // contains a NULL import descriptor.  It is used to terminate the
> imports
> +  // from a specific DLL.
> +  NewArchiveMember createNullImportDescriptor(std::vector<uint8_t>
> &Buffer);
> +
> +  // Create a NULL Thunk Entry.  This is a small object file which
> contains a
> +  // NULL Import Address Table entry and a NULL Import Lookup Table
> Entry.  It
> +  // is used to terminate the IAT and ILT.
> +  NewArchiveMember createNullThunk(std::vector<uint8_t> &Buffer);
> +
> +  // Create a short import file which is described in PE/COFF spec 7.
> Import
> +  // Library Format.
> +  NewArchiveMember createShortImport(StringRef Sym, uint16_t Ordinal,
> +                                     ImportType Type, ImportNameType
> NameType);
> +};
> +} // namespace
> +
> +NewArchiveMember
> +ObjectFactory::createImportDescriptor(std::vector<uint8_t> &Buffer) {
> +  static const uint32_t NumberOfSections = 2;
> +  static const uint32_t NumberOfSymbols = 7;
> +  static const uint32_t NumberOfRelocations = 3;
> +
> +  // COFF Header
> +  coff_file_header Header{
> +      u16(Machine),
> +      u16(NumberOfSections),
> +      u32(0),
> +      u32(sizeof(Header) + (NumberOfSections * sizeof(coff_section)) +
> +          // .idata$2
> +          sizeof(coff_import_directory_table_entry) +
> +          NumberOfRelocations * sizeof(coff_relocation) +
> +          // .idata$4
> +          (DLLName.size() + 1)),
> +      u32(NumberOfSymbols),
> +      u16(0),
> +      u16(is32bit(Machine) ? IMAGE_FILE_32BIT_MACHINE : 0),
> +  };
> +  append(Buffer, Header);
> +
> +  // Section Header Table
> +  static const coff_section SectionTable[NumberOfSections] = {
> +      {{'.', 'i', 'd', 'a', 't', 'a', '$', '2'},
> +       u32(0),
> +       u32(0),
> +       u32(sizeof(coff_import_directory_table_entry)),
> +       u32(sizeof(coff_file_header) + NumberOfSections *
> sizeof(coff_section)),
> +       u32(sizeof(coff_file_header) + NumberOfSections *
> sizeof(coff_section) +
> +           sizeof(coff_import_directory_table_entry)),
> +       u32(0),
> +       u16(NumberOfRelocations),
> +       u16(0),
> +       u32(IMAGE_SCN_ALIGN_4BYTES | IMAGE_SCN_CNT_INITIALIZED_DATA |
> +           IMAGE_SCN_MEM_READ | IMAGE_SCN_MEM_WRITE)},
> +      {{'.', 'i', 'd', 'a', 't', 'a', '$', '6'},
> +       u32(0),
> +       u32(0),
> +       u32(DLLName.size() + 1),
> +       u32(sizeof(coff_file_header) + NumberOfSections *
> sizeof(coff_section) +
> +           sizeof(coff_import_directory_table_entry) +
> +           NumberOfRelocations * sizeof(coff_relocation)),
> +       u32(0),
> +       u32(0),
> +       u16(0),
> +       u16(0),
> +       u32(IMAGE_SCN_ALIGN_2BYTES | IMAGE_SCN_CNT_INITIALIZED_DATA |
> +           IMAGE_SCN_MEM_READ | IMAGE_SCN_MEM_WRITE)},
> +  };
> +  append(Buffer, SectionTable);
> +
> +  // .idata$2
> +  static const coff_import_directory_table_entry ImportDescriptor{
> +      u32(0), u32(0), u32(0), u32(0), u32(0),
> +  };
> +  append(Buffer, ImportDescriptor);
> +
> +  static const coff_relocation RelocationTable[NumberOfRelocations] = {
> +      {u32(offsetof(coff_import_directory_table_entry, NameRVA)), u32(2),
> +       u16(getImgRelRelocation(Machine))},
> +      {u32(offsetof(coff_import_directory_table_entry,
> ImportLookupTableRVA)),
> +       u32(3), u16(getImgRelRelocation(Machine))},
> +      {u32(offsetof(coff_import_directory_table_entry,
> ImportAddressTableRVA)),
> +       u32(4), u16(getImgRelRelocation(Machine))},
> +  };
> +  append(Buffer, RelocationTable);
> +
> +  // .idata$6
> +  auto S = Buffer.size();
> +  Buffer.resize(S + DLLName.size() + 1);
> +  memcpy(&Buffer[S], DLLName.data(), DLLName.size());
> +  Buffer[S + DLLName.size()] = '\0';
> +
> +  // Symbol Table
> +  coff_symbol16 SymbolTable[NumberOfSymbols] = {
> +      {{{0, 0, 0, 0, 0, 0, 0, 0}},
> +       u32(0),
> +       u16(1),
> +       u16(0),
> +       IMAGE_SYM_CLASS_EXTERNAL,
> +       0},
> +      {{{'.', 'i', 'd', 'a', 't', 'a', '$', '2'}},
> +       u32(0),
> +       u16(1),
> +       u16(0),
> +       IMAGE_SYM_CLASS_SECTION,
> +       0},
> +      {{{'.', 'i', 'd', 'a', 't', 'a', '$', '6'}},
> +       u32(0),
> +       u16(2),
> +       u16(0),
> +       IMAGE_SYM_CLASS_STATIC,
> +       0},
> +      {{{'.', 'i', 'd', 'a', 't', 'a', '$', '4'}},
> +       u32(0),
> +       u16(0),
> +       u16(0),
> +       IMAGE_SYM_CLASS_SECTION,
> +       0},
> +      {{{'.', 'i', 'd', 'a', 't', 'a', '$', '5'}},
> +       u32(0),
> +       u16(0),
> +       u16(0),
> +       IMAGE_SYM_CLASS_SECTION,
> +       0},
> +      {{{0, 0, 0, 0, 0, 0, 0, 0}},
> +       u32(0),
> +       u16(0),
> +       u16(0),
> +       IMAGE_SYM_CLASS_EXTERNAL,
> +       0},
> +      {{{0, 0, 0, 0, 0, 0, 0, 0}},
> +       u32(0),
> +       u16(0),
> +       u16(0),
> +       IMAGE_SYM_CLASS_EXTERNAL,
> +       0},
> +  };
> +  reinterpret_cast<StringTableOffset &>(SymbolTable[0].Name).Offset =
> +      sizeof(uint32_t);
> +  reinterpret_cast<StringTableOffset &>(SymbolTable[5].Name).Offset =
> +      sizeof(uint32_t) + ImportDescriptorSymbolName.length() + 1;
> +  reinterpret_cast<StringTableOffset &>(SymbolTable[6].Name).Offset =
> +      sizeof(uint32_t) + ImportDescriptorSymbolName.length() + 1 +
> +      NullImportDescriptorSymbolName.length() + 1;
> +  append(Buffer, SymbolTable);
> +
> +  // String Table
> +  writeStringTable(Buffer,
> +                   {ImportDescriptorSymbolName,
> NullImportDescriptorSymbolName,
> +                    NullThunkSymbolName});
> +
> +  StringRef F{reinterpret_cast<const char *>(Buffer.data()),
> Buffer.size()};
> +  return {MemoryBufferRef(F, DLLName)};
> +}
> +
> +NewArchiveMember
> +ObjectFactory::createNullImportDescriptor(std::vector<uint8_t> &Buffer) {
> +  static const uint32_t NumberOfSections = 1;
> +  static const uint32_t NumberOfSymbols = 1;
> +
> +  // COFF Header
> +  coff_file_header Header{
> +      u16(Machine),
> +      u16(NumberOfSections),
> +      u32(0),
> +      u32(sizeof(Header) + (NumberOfSections * sizeof(coff_section)) +
> +          // .idata$3
> +          sizeof(coff_import_directory_table_entry)),
> +      u32(NumberOfSymbols),
> +      u16(0),
> +      u16(is32bit(Machine) ? IMAGE_FILE_32BIT_MACHINE : 0),
> +  };
> +  append(Buffer, Header);
> +
> +  // Section Header Table
> +  static const coff_section SectionTable[NumberOfSections] = {
> +      {{'.', 'i', 'd', 'a', 't', 'a', '$', '3'},
> +       u32(0),
> +       u32(0),
> +       u32(sizeof(coff_import_directory_table_entry)),
> +       u32(sizeof(coff_file_header) +
> +           (NumberOfSections * sizeof(coff_section))),
> +       u32(0),
> +       u32(0),
> +       u16(0),
> +       u16(0),
> +       u32(IMAGE_SCN_ALIGN_4BYTES | IMAGE_SCN_CNT_INITIALIZED_DATA |
> +           IMAGE_SCN_MEM_READ | IMAGE_SCN_MEM_WRITE)},
> +  };
> +  append(Buffer, SectionTable);
> +
> +  // .idata$3
> +  static const coff_import_directory_table_entry ImportDescriptor{
> +      u32(0), u32(0), u32(0), u32(0), u32(0),
> +  };
> +  append(Buffer, ImportDescriptor);
> +
> +  // Symbol Table
> +  coff_symbol16 SymbolTable[NumberOfSymbols] = {
> +      {{{0, 0, 0, 0, 0, 0, 0, 0}},
> +       u32(0),
> +       u16(1),
> +       u16(0),
> +       IMAGE_SYM_CLASS_EXTERNAL,
> +       0},
> +  };
> +  reinterpret_cast<StringTableOffset &>(SymbolTable[0].Name).Offset =
> +      sizeof(uint32_t);
> +  append(Buffer, SymbolTable);
> +
> +  // String Table
> +  writeStringTable(Buffer, {NullImportDescriptorSymbolName});
> +
> +  StringRef F{reinterpret_cast<const char *>(Buffer.data()),
> Buffer.size()};
> +  return {MemoryBufferRef(F, DLLName)};
> +}
> +
> +NewArchiveMember ObjectFactory::createNullThunk(std::vector<uint8_t>
> &Buffer) {
> +  static const uint32_t NumberOfSections = 2;
> +  static const uint32_t NumberOfSymbols = 1;
> +  uint32_t VASize = is32bit(Machine) ? 4 : 8;
> +
> +  // COFF Header
> +  coff_file_header Header{
> +      u16(Machine),
> +      u16(NumberOfSections),
> +      u32(0),
> +      u32(sizeof(Header) + (NumberOfSections * sizeof(coff_section)) +
> +          // .idata$5
> +          VASize +
> +          // .idata$4
> +          VASize),
> +      u32(NumberOfSymbols),
> +      u16(0),
> +      u16(is32bit(Machine) ? IMAGE_FILE_32BIT_MACHINE : 0),
> +  };
> +  append(Buffer, Header);
> +
> +  // Section Header Table
> +  static const coff_section SectionTable[NumberOfSections] = {
> +      {{'.', 'i', 'd', 'a', 't', 'a', '$', '5'},
> +       u32(0),
> +       u32(0),
> +       u32(VASize),
> +       u32(sizeof(coff_file_header) + NumberOfSections *
> sizeof(coff_section)),
> +       u32(0),
> +       u32(0),
> +       u16(0),
> +       u16(0),
> +       u32((is32bit(Machine) ? IMAGE_SCN_ALIGN_4BYTES
> +                             : IMAGE_SCN_ALIGN_8BYTES) |
> +           IMAGE_SCN_CNT_INITIALIZED_DATA | IMAGE_SCN_MEM_READ |
> +           IMAGE_SCN_MEM_WRITE)},
> +      {{'.', 'i', 'd', 'a', 't', 'a', '$', '4'},
> +       u32(0),
> +       u32(0),
> +       u32(VASize),
> +       u32(sizeof(coff_file_header) + NumberOfSections *
> sizeof(coff_section) +
> +           VASize),
> +       u32(0),
> +       u32(0),
> +       u16(0),
> +       u16(0),
> +       u32((is32bit(Machine) ? IMAGE_SCN_ALIGN_4BYTES
> +                             : IMAGE_SCN_ALIGN_8BYTES) |
> +           IMAGE_SCN_CNT_INITIALIZED_DATA | IMAGE_SCN_MEM_READ |
> +           IMAGE_SCN_MEM_WRITE)},
> +  };
> +  append(Buffer, SectionTable);
> +
> +  // .idata$5, ILT
> +  append(Buffer, u32(0));
> +  if (!is32bit(Machine))
> +    append(Buffer, u32(0));
> +
> +  // .idata$4, IAT
> +  append(Buffer, u32(0));
> +  if (!is32bit(Machine))
> +    append(Buffer, u32(0));
> +
> +  // Symbol Table
> +  coff_symbol16 SymbolTable[NumberOfSymbols] = {
> +      {{{0, 0, 0, 0, 0, 0, 0, 0}},
> +       u32(0),
> +       u16(1),
> +       u16(0),
> +       IMAGE_SYM_CLASS_EXTERNAL,
> +       0},
> +  };
> +  reinterpret_cast<StringTableOffset &>(SymbolTable[0].Name).Offset =
> +      sizeof(uint32_t);
> +  append(Buffer, SymbolTable);
> +
> +  // String Table
> +  writeStringTable(Buffer, {NullThunkSymbolName});
> +
> +  StringRef F{reinterpret_cast<const char *>(Buffer.data()),
> Buffer.size()};
> +  return {MemoryBufferRef{F, DLLName}};
> +}
> +
> +NewArchiveMember ObjectFactory::createShortImport(StringRef Sym,
> +                                                  uint16_t Ordinal,
> +                                                  ImportType ImportType,
> +                                                  ImportNameType
> NameType) {
> +  size_t ImpSize = DLLName.size() + Sym.size() + 2; // +2 for NULs
> +  size_t Size = sizeof(coff_import_header) + ImpSize;
> +  char *Buf = Alloc.Allocate<char>(Size);
> +  memset(Buf, 0, Size);
> +  char *P = Buf;
> +
> +  // Write short import library.
> +  auto *Imp = reinterpret_cast<coff_import_header *>(P);
> +  P += sizeof(*Imp);
> +  Imp->Sig2 = 0xFFFF;
> +  Imp->Machine = Machine;
> +  Imp->SizeOfData = ImpSize;
> +  if (Ordinal > 0)
> +    Imp->OrdinalHint = Ordinal;
> +  Imp->TypeInfo = (NameType << 2) | ImportType;
> +
> +  // Write symbol name and DLL name.
> +  memcpy(P, Sym.data(), Sym.size());
> +  P += Sym.size() + 1;
> +  memcpy(P, DLLName.data(), DLLName.size());
> +
> +  return {MemoryBufferRef(StringRef(Buf, Size), DLLName)};
> +}
> +
> +std::error_code writeImportLibrary(StringRef DLLName, StringRef Path,
> +                                   ArrayRef<COFFShortExport> Exports,
> +                                   MachineTypes Machine) {
> +
> +  std::vector<NewArchiveMember> Members;
> +  ObjectFactory OF(llvm::sys::path::filename(DLLName), Machine);
> +
> +  std::vector<uint8_t> ImportDescriptor;
> +  Members.push_back(OF.createImportDescriptor(ImportDescriptor));
> +
> +  std::vector<uint8_t> NullImportDescriptor;
> +  Members.push_back(OF.createNullImportDescriptor(NullImportDescriptor));
> +
> +  std::vector<uint8_t> NullThunk;
> +  Members.push_back(OF.createNullThunk(NullThunk));
> +
> +  for (COFFShortExport E : Exports) {
> +    if (E.Private)
> +      continue;
> +
> +    ImportType ImportType = IMPORT_CODE;
> +    if (E.Data)
> +      ImportType = IMPORT_DATA;
> +    if (E.Constant)
> +      ImportType = IMPORT_CONST;
> +
> +    StringRef SymbolName = E.isWeak() ? E.ExtName : E.Name;
> +    ImportNameType NameType = getNameType(SymbolName, E.Name, Machine);
> +    Expected<std::string> Name = E.ExtName.empty()
> +                                     ? SymbolName
> +                                     : replace(SymbolName, E.Name,
> E.ExtName);
> +
> +    if (!Name) {
> +      return errorToErrorCode(Name.takeError());
> +    }
> +
> +    Members.push_back(
> +        OF.createShortImport(*Name, E.Ordinal, ImportType, NameType));
> +  }
> +
> +  std::pair<StringRef, std::error_code> Result =
> +      writeArchive(Path, Members, /*WriteSymtab*/ true,
> object::Archive::K_GNU,
> +                   /*Deterministic*/ true, /*Thin*/ false);
> +
> +  return Result.second;
> +}
> +
> +} // namespace object
> +} // namespace llvm
>
> Added: llvm/trunk/lib/Object/COFFModuleDefinition.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Object/
> COFFModuleDefinition.cpp?rev=303490&view=auto
> ============================================================
> ==================
> --- llvm/trunk/lib/Object/COFFModuleDefinition.cpp (added)
> +++ llvm/trunk/lib/Object/COFFModuleDefinition.cpp Sat May 20 14:56:29
> 2017
> @@ -0,0 +1,319 @@
> +//===--- COFFModuleDefinition.cpp - Simple DEF parser
> ---------------------===//
> +//
> +//                     The LLVM Compiler Infrastructure
> +//
> +// This file is distributed under the University of Illinois Open Source
> +// License. See LICENSE.TXT for details.
> +//
> +//===------------------------------------------------------
> ----------------===//
> +//
> +// Windows-specific.
> +// A parser for the module-definition file (.def file).
> +//
> +// The format of module-definition files are described in this document:
> +// https://msdn.microsoft.com/en-us/library/28d6s79h.aspx
> +//
> +//===------------------------------------------------------
> ----------------===//
> +
> +#include "llvm/Object/COFFModuleDefinition.h"
> +#include "llvm/ADT/StringRef.h"
> +#include "llvm/ADT/StringSwitch.h"
> +#include "llvm/Object/COFF.h"
> +#include "llvm/Object/COFFImportFile.h"
> +#include "llvm/Object/Error.h"
> +#include "llvm/Support/Error.h"
> +#include "llvm/Support/raw_ostream.h"
> +
> +using namespace llvm::COFF;
> +using namespace llvm;
> +
> +namespace llvm {
> +namespace object {
> +
> +enum Kind {
> +  Unknown,
> +  Eof,
> +  Identifier,
> +  Comma,
> +  Equal,
> +  KwBase,
> +  KwConstant,
> +  KwData,
> +  KwExports,
> +  KwHeapsize,
> +  KwLibrary,
> +  KwName,
> +  KwNoname,
> +  KwPrivate,
> +  KwStacksize,
> +  KwVersion,
> +};
> +
> +struct Token {
> +  explicit Token(Kind T = Unknown, StringRef S = "") : K(T), Value(S) {}
> +  Kind K;
> +  StringRef Value;
> +};
> +
> +static bool isDecorated(StringRef Sym) {
> +  return Sym.startswith("_") || Sym.startswith("@") ||
> Sym.startswith("?");
> +}
> +
> +static Error createError(const Twine &Err) {
> +  return make_error<StringError>(Err.getSingleStringRef(),
> +                                 object_error::parse_failed);
> +}
> +
> +class Lexer {
> +public:
> +  Lexer(StringRef S) : Buf(S) {}
> +
> +  Token lex() {
> +    Buf = Buf.trim();
> +    if (Buf.empty())
> +      return Token(Eof);
> +
> +    switch (Buf[0]) {
> +    case '\0':
> +      return Token(Eof);
> +    case ';': {
> +      size_t End = Buf.find('\n');
> +      Buf = (End == Buf.npos) ? "" : Buf.drop_front(End);
> +      return lex();
> +    }
> +    case '=':
> +      Buf = Buf.drop_front();
> +      return Token(Equal, "=");
> +    case ',':
> +      Buf = Buf.drop_front();
> +      return Token(Comma, ",");
> +    case '"': {
> +      StringRef S;
> +      std::tie(S, Buf) = Buf.substr(1).split('"');
> +      return Token(Identifier, S);
> +    }
> +    default: {
> +      size_t End = Buf.find_first_of("=,\r\n \t\v");
> +      StringRef Word = Buf.substr(0, End);
> +      Kind K = llvm::StringSwitch<Kind>(Word)
> +                   .Case("BASE", KwBase)
> +                   .Case("CONSTANT", KwConstant)
> +                   .Case("DATA", KwData)
> +                   .Case("EXPORTS", KwExports)
> +                   .Case("HEAPSIZE", KwHeapsize)
> +                   .Case("LIBRARY", KwLibrary)
> +                   .Case("NAME", KwName)
> +                   .Case("NONAME", KwNoname)
> +                   .Case("PRIVATE", KwPrivate)
> +                   .Case("STACKSIZE", KwStacksize)
> +                   .Case("VERSION", KwVersion)
> +                   .Default(Identifier);
> +      Buf = (End == Buf.npos) ? "" : Buf.drop_front(End);
> +      return Token(K, Word);
> +    }
> +    }
> +  }
> +
> +private:
> +  StringRef Buf;
> +};
> +
> +class Parser {
> +public:
> +  explicit Parser(StringRef S, MachineTypes M) : Lex(S), Machine(M) {}
> +
> +  Expected<COFFModuleDefinition> parse() {
> +    do {
> +      if (Error Err = parseOne())
> +        return std::move(Err);
> +    } while (Tok.K != Eof);
> +    return Info;
> +  }
> +
> +private:
> +  void read() {
> +    if (Stack.empty()) {
> +      Tok = Lex.lex();
> +      return;
> +    }
> +    Tok = Stack.back();
> +    Stack.pop_back();
> +  }
> +
> +  Error readAsInt(uint64_t *I) {
> +    read();
> +    if (Tok.K != Identifier || Tok.Value.getAsInteger(10, *I))
> +      return createError("integer expected");
> +    return Error::success();
> +  }
> +
> +  Error expect(Kind Expected, StringRef Msg) {
> +    read();
> +    if (Tok.K != Expected)
> +      return createError(Msg);
> +    return Error::success();
> +  }
> +
> +  void unget() { Stack.push_back(Tok); }
> +
> +  Error parseOne() {
> +    read();
> +    switch (Tok.K) {
> +    case Eof:
> +      return Error::success();
> +    case KwExports:
> +      for (;;) {
> +        read();
> +        if (Tok.K != Identifier) {
> +          unget();
> +          return Error::success();
> +        }
> +        if (Error Err = parseExport())
> +          return Err;
> +      }
> +    case KwHeapsize:
> +      return parseNumbers(&Info.HeapReserve, &Info.HeapCommit);
> +    case KwStacksize:
> +      return parseNumbers(&Info.StackReserve, &Info.StackCommit);
> +    case KwLibrary:
> +    case KwName: {
> +      bool IsDll = Tok.K == KwLibrary; // Check before parseName.
> +      std::string Name;
> +      if (Error Err = parseName(&Name, &Info.ImageBase))
> +        return Err;
> +      // Append the appropriate file extension if not already present.
> +      StringRef Ext = IsDll ? ".dll" : ".exe";
> +      if (!StringRef(Name).endswith_lower(Ext))
> +        Name += Ext;
> +
> +      // Set the output file, but don't override /out if it was already
> passed.
> +      if (Info.OutputFile.empty())
> +        Info.OutputFile = Name;
> +      return Error::success();
> +    }
> +    case KwVersion:
> +      return parseVersion(&Info.MajorImageVersion,
> &Info.MinorImageVersion);
> +    default:
> +      return createError("unknown directive: " + Tok.Value);
> +    }
> +  }
> +
> +  Error parseExport() {
> +    COFFShortExport E;
> +    E.Name = Tok.Value;
> +    read();
> +    if (Tok.K == Equal) {
> +      read();
> +      if (Tok.K != Identifier)
> +        return createError("identifier expected, but got " + Tok.Value);
> +      E.ExtName = E.Name;
> +      E.Name = Tok.Value;
> +    } else {
> +      unget();
> +    }
> +
> +    if (Machine == IMAGE_FILE_MACHINE_I386) {
> +      if (!isDecorated(E.Name))
> +        E.Name = (std::string("_").append(E.Name));
> +      if (!E.ExtName.empty() && !isDecorated(E.ExtName))
> +        E.ExtName = (std::string("_").append(E.ExtName));
> +    }
> +
> +    for (;;) {
> +      read();
> +      if (Tok.K == Identifier && Tok.Value[0] == '@') {
> +        Tok.Value.drop_front().getAsInteger(10, E.Ordinal);
> +        read();
> +        if (Tok.K == KwNoname) {
> +          E.Noname = true;
> +        } else {
> +          unget();
> +        }
> +        continue;
> +      }
> +      if (Tok.K == KwData) {
> +        E.Data = true;
> +        continue;
> +      }
> +      if (Tok.K == KwConstant) {
> +        E.Constant = true;
> +        continue;
> +      }
> +      if (Tok.K == KwPrivate) {
> +        E.Private = true;
> +        continue;
> +      }
> +      unget();
> +      Info.Exports.push_back(E);
> +      return Error::success();
> +    }
> +  }
> +
> +  // HEAPSIZE/STACKSIZE reserve[,commit]
> +  Error parseNumbers(uint64_t *Reserve, uint64_t *Commit) {
> +    if (Error Err = readAsInt(Reserve))
> +      return Err;
> +    read();
> +    if (Tok.K != Comma) {
> +      unget();
> +      Commit = nullptr;
> +      return Error::success();
> +    }
> +    if (Error Err = readAsInt(Commit))
> +      return Err;
> +    return Error::success();
> +  }
> +
> +  // NAME outputPath [BASE=address]
> +  Error parseName(std::string *Out, uint64_t *Baseaddr) {
> +    read();
> +    if (Tok.K == Identifier) {
> +      *Out = Tok.Value;
> +    } else {
> +      *Out = "";
> +      unget();
> +      return Error::success();
> +    }
> +    read();
> +    if (Tok.K == KwBase) {
> +      if (Error Err = expect(Equal, "'=' expected"))
> +        return Err;
> +      if (Error Err = readAsInt(Baseaddr))
> +        return Err;
> +    } else {
> +      unget();
> +      *Baseaddr = 0;
> +    }
> +    return Error::success();
> +  }
> +
> +  // VERSION major[.minor]
> +  Error parseVersion(uint32_t *Major, uint32_t *Minor) {
> +    read();
> +    if (Tok.K != Identifier)
> +      return createError("identifier expected, but got " + Tok.Value);
> +    StringRef V1, V2;
> +    std::tie(V1, V2) = Tok.Value.split('.');
> +    if (V1.getAsInteger(10, *Major))
> +      return createError("integer expected, but got " + Tok.Value);
> +    if (V2.empty())
> +      *Minor = 0;
> +    else if (V2.getAsInteger(10, *Minor))
> +      return createError("integer expected, but got " + Tok.Value);
> +    return Error::success();
> +  }
> +
> +  Lexer Lex;
> +  Token Tok;
> +  std::vector<Token> Stack;
> +  MachineTypes Machine;
> +  COFFModuleDefinition Info;
> +};
> +
> +Expected<COFFModuleDefinition> parseCOFFModuleDefinition(MemoryBufferRef
> MB,
> +                                                         MachineTypes
> Machine) {
> +  return Parser(MB.getBuffer(), Machine).parse();
> +}
> +
> +} // namespace object
> +} // namespace llvm
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170531/6c114a31/attachment-0001.html>


More information about the llvm-commits mailing list