[lld] r225764 - Convert other drivers to use WrapperNode.
Jean-Daniel Dupas
dev at xenonium.com
Tue Jan 13 14:17:28 PST 2015
I got it. The issue is in loadIndirectDylib().
DarwinLinkingContext was using MachFileNode to parse reexported dylib. The FileNode was responsible to handle fat binaries.
By removing the MachOFileNode, you also removed the fat binary support.
All code paths that was using MachOFileNode should be review to make sure we properly call sliceFromFatFile() on the parsed buffer.
> Le 13 janv. 2015 à 22:52, Rui Ueyama <ruiu at google.com> a écrit :
>
> Thanks. I'll try that on a Mac. Wondering why this basic functionality breakage wasn't caught by the tests.
>
> On Tue, Jan 13, 2015 at 1:49 PM, Jean-Daniel Dupas <dev at xenonium.com> wrote:
> If you have a Mac under the hand, linking a simple hello world should reproduce the issue.
>
> —— main.c ——
> #include <stdio.h>
>
> int main(int argc, const char * argv[]) {
> printf("Hello, World!\n");
> return 0;
> }
> ————
> clang -c main.c
> lld -flavor darwin -o hello main.o -lSystem
>
> Assertion failed: (dylib.file), function exports, file /Users/jddupas/Projects/OpenSource/llvm/llvm/tools/lld/lib/ReaderWriter/MachO/File.h, line 305.
> 0 lld 0x0000000102840039 llvm::sys::PrintStackTrace(__sFILE*) + 57
> 1 lld 0x000000010284079b SignalHandler(int) + 635
> 2 libsystem_platform.dylib 0x00007fff924def1a _sigtramp + 26
> 3 libsystem_platform.dylib 0x00007fff607dc764 _sigtramp + 3459242084
> 4 lld 0x0000000102840506 abort + 22
> 5 lld 0x00000001028404e1 __assert_rtn + 81
> 6 lld 0x00000001025dd946 lld::mach_o::MachODylibFile::exports(llvm::StringRef, llvm::StringRef) const + 278
> 7 lld 0x00000001026c000e std::__1::__function::__func<lld::Resolver::handleSharedLibrary(lld::File const&)::$_1, std::__1::allocator<lld::Resolver::handleSharedLibrary(lld::File const&)::$_1>, void (llvm::StringRef, bool)>::operator()(llvm::StringRef&&, bool&&) + 46
> 8 lld 0x00000001026bdab0 lld::Resolver::forEachUndefines(bool, std::__1::function<void (llvm::StringRef, bool)>) + 144
> 9 lld 0x00000001026bdd51 lld::Resolver::handleSharedLibrary(lld::File const&) + 113
> …
>
> > Le 13 janv. 2015 à 22:37, Rui Ueyama <ruiu at google.com> a écrit :
> >
> > I'm sorry for the breakage. How can I reproduce the issue?
> >
> > On Tue, Jan 13, 2015 at 1:35 PM, Jean-Daniel Dupas <dev at xenonium.com> wrote:
> > Looks like this commit broke the Mach-O driver.
> > I didn’t investigate yet but I encounter assertion with reexported dyld that are not loaded.
> >
> > Assertion failed: (dylib.file), function exports, file /Users/jddupas/Projects/OpenSource/llvm/llvm/tools/lld/lib/ReaderWriter/MachO/File.h, line 309
> >
> >
> > > Le 13 janv. 2015 à 05:33, Rui Ueyama <ruiu at google.com> a écrit :
> > >
> > > Author: ruiu
> > > Date: Mon Jan 12 22:33:07 2015
> > > New Revision: 225764
> > >
> > > URL: http://llvm.org/viewvc/llvm-project?rev=225764&view=rev
> > > Log:
> > > Convert other drivers to use WrapperNode.
> > >
> > > Removed:
> > > lld/trunk/include/lld/Driver/DarwinInputGraph.h
> > > lld/trunk/include/lld/Driver/GnuLdInputGraph.h
> > > lld/trunk/include/lld/Driver/WinLinkInputGraph.h
> > > lld/trunk/lib/Driver/DarwinInputGraph.cpp
> > > lld/trunk/lib/Driver/GnuLdInputGraph.cpp
> > > lld/trunk/lib/Driver/WinLinkInputGraph.cpp
> > > lld/trunk/test/Driver/libsearch-inputGraph.test
> > > Modified:
> > > lld/trunk/docs/Readers.rst
> > > lld/trunk/include/lld/Core/ArchiveLibraryFile.h
> > > lld/trunk/include/lld/Core/File.h
> > > lld/trunk/include/lld/Driver/Driver.h
> > > lld/trunk/include/lld/ReaderWriter/ELFLinkingContext.h
> > > lld/trunk/include/lld/ReaderWriter/MachOLinkingContext.h
> > > lld/trunk/include/lld/ReaderWriter/Reader.h
> > > lld/trunk/lib/Driver/CMakeLists.txt
> > > lld/trunk/lib/Driver/CoreDriver.cpp
> > > lld/trunk/lib/Driver/DarwinLdDriver.cpp
> > > lld/trunk/lib/Driver/Driver.cpp
> > > lld/trunk/lib/Driver/GnuLdDriver.cpp
> > > lld/trunk/lib/Driver/WinLinkDriver.cpp
> > > lld/trunk/lib/Passes/RoundTripNativePass.cpp
> > > lld/trunk/lib/Passes/RoundTripYAMLPass.cpp
> > > lld/trunk/lib/ReaderWriter/ELF/ELFReader.h
> > > lld/trunk/lib/ReaderWriter/ELF/Mips/MipsELFReader.h
> > > lld/trunk/lib/ReaderWriter/FileArchive.cpp
> > > lld/trunk/lib/ReaderWriter/MachO/ExecutableAtoms.hpp
> > > lld/trunk/lib/ReaderWriter/MachO/MachOLinkingContext.cpp
> > > lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFileBinaryReader.cpp
> > > lld/trunk/lib/ReaderWriter/Native/ReaderNative.cpp
> > > lld/trunk/lib/ReaderWriter/PECOFF/LinkerGeneratedSymbolFile.h
> > > lld/trunk/lib/ReaderWriter/PECOFF/ReaderCOFF.cpp
> > > lld/trunk/lib/ReaderWriter/PECOFF/ReaderImportHeader.cpp
> > > lld/trunk/lib/ReaderWriter/Reader.cpp
> > > lld/trunk/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp
> > > lld/trunk/test/pecoff/hello64.test
> > > lld/trunk/unittests/DriverTests/WinLinkDriverTest.cpp
> > >
> > > Modified: lld/trunk/docs/Readers.rst
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/docs/Readers.rst?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/docs/Readers.rst (original)
> > > +++ lld/trunk/docs/Readers.rst Mon Jan 12 22:33:07 2015
> > > @@ -66,7 +66,7 @@ Readers are factories
> > > ---------------------
> > >
> > > The linker will usually only instantiate your Reader once. That one Reader will
> > > -have its parseFile() method called many times with different input files.
> > > +have its loadFile() method called many times with different input files.
> > > To support multithreaded linking, the Reader may be parsing multiple input
> > > files in parallel. Therefore, there should be no parsing state in you Reader
> > > object. Any parsing state should be in ivars of your File subclass or in
> > > @@ -74,8 +74,8 @@ some temporary object.
> > >
> > > The key method to implement in a reader is::
> > >
> > > - virtual error_code parseFile(LinkerInput &input,
> > > - std::vector<std::unique_ptr<File>> &result);
> > > + virtual error_code loadFile(LinkerInput &input,
> > > + std::vector<std::unique_ptr<File>> &result);
> > >
> > > It takes a memory buffer (which contains the contents of the object file
> > > being read) and returns an instantiated lld::File object which is
> > >
> > > Modified: lld/trunk/include/lld/Core/ArchiveLibraryFile.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/Core/ArchiveLibraryFile.h?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/include/lld/Core/ArchiveLibraryFile.h (original)
> > > +++ lld/trunk/include/lld/Core/ArchiveLibraryFile.h Mon Jan 12 22:33:07 2015
> > > @@ -35,7 +35,7 @@ public:
> > > virtual const File *find(StringRef name, bool dataSymbolOnly) const = 0;
> > >
> > > virtual std::error_code
> > > - parseAllMembers(std::vector<std::unique_ptr<File>> &result) const = 0;
> > > + parseAllMembers(std::vector<std::unique_ptr<File>> &result) = 0;
> > >
> > > /// Returns a set of all defined symbols in the archive, i.e. all
> > > /// resolvable symbol using this file.
> > >
> > > Modified: lld/trunk/include/lld/Core/File.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/Core/File.h?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/include/lld/Core/File.h (original)
> > > +++ lld/trunk/include/lld/Core/File.h Mon Jan 12 22:33:07 2015
> > > @@ -274,7 +274,8 @@ protected:
> > > /// can do unit testing a driver using non-existing file paths.
> > > class ErrorFile : public File {
> > > public:
> > > - ErrorFile(StringRef p, std::error_code ec) : File(p, kindObject), _ec(ec) {}
> > > + ErrorFile(StringRef path, std::error_code ec)
> > > + : File(path, kindObject), _ec(ec) {}
> > >
> > > std::error_code doParse() override { return _ec; }
> > >
> > >
> > > Removed: lld/trunk/include/lld/Driver/DarwinInputGraph.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/Driver/DarwinInputGraph.h?rev=225763&view=auto
> > > ==============================================================================
> > > --- lld/trunk/include/lld/Driver/DarwinInputGraph.h (original)
> > > +++ lld/trunk/include/lld/Driver/DarwinInputGraph.h (removed)
> > > @@ -1,56 +0,0 @@
> > > -//===- lld/Driver/DarwinInputGraph.h - Input Graph Node for Mach-O linker -===//
> > > -//
> > > -// The LLVM Linker
> > > -//
> > > -// This file is distributed under the University of Illinois Open Source
> > > -// License. See LICENSE.TXT for details.
> > > -//
> > > -//===----------------------------------------------------------------------===//
> > > -///
> > > -/// \file
> > > -///
> > > -/// Handles Options for MachO linking and provides InputElements
> > > -/// for MachO linker
> > > -///
> > > -//===----------------------------------------------------------------------===//
> > > -
> > > -#ifndef LLD_DRIVER_DARWIN_INPUT_GRAPH_H
> > > -#define LLD_DRIVER_DARWIN_INPUT_GRAPH_H
> > > -
> > > -#include "lld/Core/ArchiveLibraryFile.h"
> > > -#include "lld/Core/InputGraph.h"
> > > -#include "lld/ReaderWriter/MachOLinkingContext.h"
> > > -
> > > -namespace lld {
> > > -
> > > -/// \brief Represents a MachO File
> > > -class MachOFileNode : public FileNode {
> > > -public:
> > > - MachOFileNode(StringRef path, MachOLinkingContext &ctx)
> > > - : FileNode(path), _context(ctx), _isWholeArchive(false),
> > > - _upwardDylib(false) {}
> > > -
> > > - /// \brief Parse the input file to lld::File.
> > > - std::error_code parse(const LinkingContext &ctx,
> > > - raw_ostream &diagnostics) override;
> > > -
> > > - void setLoadWholeArchive(bool value=true) {
> > > - _isWholeArchive = value;
> > > - }
> > > -
> > > - void setUpwardDylib(bool value=true) {
> > > - _upwardDylib = value;
> > > - }
> > > -
> > > -private:
> > > - void narrowFatBuffer(std::unique_ptr<MemoryBuffer> &mb, StringRef filePath);
> > > -
> > > - MachOLinkingContext &_context;
> > > - std::unique_ptr<const ArchiveLibraryFile> _archiveFile;
> > > - bool _isWholeArchive;
> > > - bool _upwardDylib;
> > > -};
> > > -
> > > -} // namespace lld
> > > -
> > > -#endif
> > >
> > > Modified: lld/trunk/include/lld/Driver/Driver.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/Driver/Driver.h?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/include/lld/Driver/Driver.h (original)
> > > +++ lld/trunk/include/lld/Driver/Driver.h Mon Jan 12 22:33:07 2015
> > > @@ -36,7 +36,7 @@ typedef std::vector<std::unique_ptr<File
> > >
> > > FileVector makeErrorFile(StringRef path, std::error_code ec);
> > > FileVector parseMemberFiles(FileVector &files);
> > > -FileVector parseFile(LinkingContext &ctx, StringRef path, bool wholeArchive);
> > > +FileVector loadFile(LinkingContext &ctx, StringRef path, bool wholeArchive);
> > >
> > > /// Base class for all Drivers.
> > > class Driver {
> > >
> > > Removed: lld/trunk/include/lld/Driver/GnuLdInputGraph.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/Driver/GnuLdInputGraph.h?rev=225763&view=auto
> > > ==============================================================================
> > > --- lld/trunk/include/lld/Driver/GnuLdInputGraph.h (original)
> > > +++ lld/trunk/include/lld/Driver/GnuLdInputGraph.h (removed)
> > > @@ -1,85 +0,0 @@
> > > -//===- lld/Driver/GnuLdInputGraph.h - Input Graph Node for ELF linker------===//
> > > -//
> > > -// The LLVM Linker
> > > -//
> > > -// This file is distributed under the University of Illinois Open Source
> > > -// License. See LICENSE.TXT for details.
> > > -//
> > > -//===----------------------------------------------------------------------===//
> > > -///
> > > -/// \file
> > > -///
> > > -/// Handles Options for the GNU style linker for ELF and provides InputElements
> > > -/// for the GNU style linker for ELF
> > > -///
> > > -//===----------------------------------------------------------------------===//
> > > -
> > > -#ifndef LLD_DRIVER_GNU_LD_INPUT_GRAPH_H
> > > -#define LLD_DRIVER_GNU_LD_INPUT_GRAPH_H
> > > -
> > > -#include "lld/Core/ArchiveLibraryFile.h"
> > > -#include "lld/Core/InputGraph.h"
> > > -#include "lld/Core/Resolver.h"
> > > -#include "lld/ReaderWriter/ELFLinkingContext.h"
> > > -
> > > -namespace lld {
> > > -
> > > -/// \brief Represents a ELF File
> > > -class ELFFileNode : public FileNode {
> > > -public:
> > > - /// \brief The attributes class provides a way for a input file to look into
> > > - /// all the positional attributes that were specified in the command line.
> > > - /// There are few positional operators and the number of arguments to the
> > > - /// ELFFileNode class keeps growing. This achieves code to be clean as well.
> > > - class Attributes {
> > > - public:
> > > - Attributes()
> > > - : _isWholeArchive(false), _asNeeded(false), _isDashlPrefix(false),
> > > - _isSysRooted(false) {}
> > > - void setWholeArchive(bool isWholeArchive) {
> > > - _isWholeArchive = isWholeArchive;
> > > - }
> > > - void setAsNeeded(bool asNeeded) { _asNeeded = asNeeded; }
> > > - void setDashlPrefix(bool isDashlPrefix) { _isDashlPrefix = isDashlPrefix; }
> > > - void setSysRooted(bool isSysRooted) { _isSysRooted = isSysRooted; }
> > > -
> > > - public:
> > > - bool _isWholeArchive;
> > > - bool _asNeeded;
> > > - bool _isDashlPrefix;
> > > - bool _isSysRooted;
> > > - };
> > > -
> > > - ELFFileNode(ELFLinkingContext &ctx, StringRef path, Attributes &attributes)
> > > - : FileNode(path), _elfLinkingContext(ctx), _attributes(attributes) {}
> > > -
> > > - ErrorOr<StringRef> getPath(const LinkingContext &ctx) const override;
> > > -
> > > - /// \brief create an error string for printing purposes
> > > - std::string errStr(std::error_code) override;
> > > -
> > > - /// \brief Dump the Input Element
> > > - bool dump(raw_ostream &diagnostics) override {
> > > - diagnostics << "Name : " << *getPath(_elfLinkingContext) << "\n"
> > > - << "Type : ELF File\n"
> > > - << "Attributes :\n"
> > > - << " - wholeArchive : "
> > > - << ((_attributes._isWholeArchive) ? "true" : "false") << "\n"
> > > - << " - asNeeded : "
> > > - << ((_attributes._asNeeded) ? "true" : "false") << "\n";
> > > - return true;
> > > - }
> > > -
> > > - /// \brief Parse the input file to lld::File.
> > > - std::error_code parse(const LinkingContext &, raw_ostream &) override;
> > > -
> > > -private:
> > > - llvm::BumpPtrAllocator _alloc;
> > > - const ELFLinkingContext &_elfLinkingContext;
> > > - std::unique_ptr<const ArchiveLibraryFile> _archiveFile;
> > > - const Attributes _attributes;
> > > -};
> > > -
> > > -} // namespace lld
> > > -
> > > -#endif
> > >
> > > Removed: lld/trunk/include/lld/Driver/WinLinkInputGraph.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/Driver/WinLinkInputGraph.h?rev=225763&view=auto
> > > ==============================================================================
> > > --- lld/trunk/include/lld/Driver/WinLinkInputGraph.h (original)
> > > +++ lld/trunk/include/lld/Driver/WinLinkInputGraph.h (removed)
> > > @@ -1,45 +0,0 @@
> > > -//===- lld/Driver/WinLinkInputGraph.h - Input Graph Node for COFF linker --===//
> > > -//
> > > -// The LLVM Linker
> > > -//
> > > -// This file is distributed under the University of Illinois Open Source
> > > -// License. See LICENSE.TXT for details.
> > > -//
> > > -//===----------------------------------------------------------------------===//
> > > -///
> > > -/// \file
> > > -///
> > > -/// Handles Options for PECOFF linking and provides InputElements
> > > -/// for PECOFF linker
> > > -///
> > > -//===----------------------------------------------------------------------===//
> > > -
> > > -#ifndef LLD_DRIVER_WIN_LINK_INPUT_GRAPH_H
> > > -#define LLD_DRIVER_WIN_LINK_INPUT_GRAPH_H
> > > -
> > > -#include "lld/Core/InputGraph.h"
> > > -#include "lld/ReaderWriter/PECOFFLinkingContext.h"
> > > -#include <map>
> > > -
> > > -namespace lld {
> > > -
> > > -/// \brief Represents a PECOFF File
> > > -class PECOFFFileNode : public FileNode {
> > > -public:
> > > - PECOFFFileNode(PECOFFLinkingContext &ctx, StringRef path)
> > > - : FileNode(path), _ctx(ctx), _parsed(false) {}
> > > -
> > > - /// \brief Parse the input file to lld::File.
> > > - std::error_code parse(const LinkingContext &ctx,
> > > - raw_ostream &diagnostics) override;
> > > -
> > > -protected:
> > > - const PECOFFLinkingContext &_ctx;
> > > -
> > > -private:
> > > - bool _parsed;
> > > -};
> > > -
> > > -} // namespace lld
> > > -
> > > -#endif
> > >
> > > Modified: lld/trunk/include/lld/ReaderWriter/ELFLinkingContext.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/ReaderWriter/ELFLinkingContext.h?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/include/lld/ReaderWriter/ELFLinkingContext.h (original)
> > > +++ lld/trunk/include/lld/ReaderWriter/ELFLinkingContext.h Mon Jan 12 22:33:07 2015
> > > @@ -290,6 +290,28 @@ public:
> > > bool alignSegments() const { return _alignSegments; }
> > > void setAlignSegments(bool align) { _alignSegments = align; }
> > >
> > > + /// \brief The attributes class provides a way for a input file to look into
> > > + /// all the positional attributes that were specified in the command line.
> > > + /// There are few positional operators and the number of arguments to the
> > > + /// ELFFileNode class keeps growing. This achieves code to be clean as well.
> > > + class Attributes {
> > > + public:
> > > + Attributes()
> > > + : _isWholeArchive(false), _asNeeded(false), _isDashlPrefix(false),
> > > + _isSysRooted(false) {}
> > > + void setWholeArchive(bool isWholeArchive) {
> > > + _isWholeArchive = isWholeArchive;
> > > + }
> > > + void setAsNeeded(bool asNeeded) { _asNeeded = asNeeded; }
> > > + void setDashlPrefix(bool isDashlPrefix) { _isDashlPrefix = isDashlPrefix; }
> > > + void setSysRooted(bool isSysRooted) { _isSysRooted = isSysRooted; }
> > > +
> > > + bool _isWholeArchive;
> > > + bool _asNeeded;
> > > + bool _isDashlPrefix;
> > > + bool _isSysRooted;
> > > + };
> > > +
> > > private:
> > > ELFLinkingContext() LLVM_DELETED_FUNCTION;
> > >
> > >
> > > Modified: lld/trunk/include/lld/ReaderWriter/MachOLinkingContext.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/ReaderWriter/MachOLinkingContext.h?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/include/lld/ReaderWriter/MachOLinkingContext.h (original)
> > > +++ lld/trunk/include/lld/ReaderWriter/MachOLinkingContext.h Mon Jan 12 22:33:07 2015
> > > @@ -350,7 +350,7 @@ private:
> > > mutable llvm::StringMap<mach_o::MachODylibFile*> _pathToDylibMap;
> > > mutable std::set<mach_o::MachODylibFile*> _allDylibs;
> > > mutable std::set<mach_o::MachODylibFile*> _upwardDylibs;
> > > - mutable std::vector<std::unique_ptr<class MachOFileNode>> _indirectDylibs;
> > > + mutable std::vector<std::unique_ptr<File>> _indirectDylibs;
> > > ExportMode _exportMode;
> > > llvm::StringSet<> _exportedSymbols;
> > > DebugInfoMode _debugInfoMode;
> > >
> > > Modified: lld/trunk/include/lld/ReaderWriter/Reader.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/include/lld/ReaderWriter/Reader.h?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/include/lld/ReaderWriter/Reader.h (original)
> > > +++ lld/trunk/include/lld/ReaderWriter/Reader.h Mon Jan 12 22:33:07 2015
> > > @@ -55,8 +55,8 @@ public:
> > > /// file) and create a File object.
> > > /// The resulting File object takes ownership of the MemoryBuffer.
> > > virtual std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > - std::vector<std::unique_ptr<File>> &result) const = 0;
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > + std::vector<std::unique_ptr<File>> &result) const = 0;
> > > };
> > >
> > >
> > > @@ -93,8 +93,8 @@ public:
> > >
> > > /// Walk the list of registered Readers and find one that can parse the
> > > /// supplied file and parse it.
> > > - std::error_code parseFile(std::unique_ptr<MemoryBuffer> mb,
> > > - std::vector<std::unique_ptr<File>> &result) const;
> > > + std::error_code loadFile(std::unique_ptr<MemoryBuffer> mb,
> > > + std::vector<std::unique_ptr<File>> &result) const;
> > >
> > > /// Walk the list of registered kind tables to convert a Reference Kind
> > > /// name to a value.
> > >
> > > Modified: lld/trunk/lib/Driver/CMakeLists.txt
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Driver/CMakeLists.txt?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/Driver/CMakeLists.txt (original)
> > > +++ lld/trunk/lib/Driver/CMakeLists.txt Mon Jan 12 22:33:07 2015
> > > @@ -12,14 +12,11 @@ add_public_tablegen_target(DriverOptions
> > >
> > > add_lld_library(lldDriver
> > > CoreDriver.cpp
> > > - DarwinInputGraph.cpp
> > > DarwinLdDriver.cpp
> > > Driver.cpp
> > > GnuLdDriver.cpp
> > > - GnuLdInputGraph.cpp
> > > UniversalDriver.cpp
> > > WinLinkDriver.cpp
> > > - WinLinkInputGraph.cpp
> > > WinLinkModuleDef.cpp
> > > )
> > >
> > >
> > > Modified: lld/trunk/lib/Driver/CoreDriver.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Driver/CoreDriver.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/Driver/CoreDriver.cpp (original)
> > > +++ lld/trunk/lib/Driver/CoreDriver.cpp Mon Jan 12 22:33:07 2015
> > > @@ -152,7 +152,7 @@ bool CoreDriver::parse(int argc, const c
> > >
> > > case OPT_INPUT: {
> > > std::vector<std::unique_ptr<File>> files
> > > - = parseFile(ctx, inputArg->getValue(), false);
> > > + = loadFile(ctx, inputArg->getValue(), false);
> > > for (std::unique_ptr<File> &file : files) {
> > > inputGraph->addInputElement(std::unique_ptr<InputElement>(
> > > new WrapperNode(std::move(file))));
> > >
> > > Removed: lld/trunk/lib/Driver/DarwinInputGraph.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Driver/DarwinInputGraph.cpp?rev=225763&view=auto
> > > ==============================================================================
> > > --- lld/trunk/lib/Driver/DarwinInputGraph.cpp (original)
> > > +++ lld/trunk/lib/Driver/DarwinInputGraph.cpp (removed)
> > > @@ -1,83 +0,0 @@
> > > -//===- lib/ReaderWriter/MachO/DarwinInputGraph.cpp ------------------------===//
> > > -//
> > > -// The LLVM Linker
> > > -//
> > > -// This file is distributed under the University of Illinois Open Source
> > > -// License. See LICENSE.TXT for details.
> > > -//
> > > -//===----------------------------------------------------------------------===//
> > > -
> > > -#include "lld/Driver/DarwinInputGraph.h"
> > > -#include "lld/Core/ArchiveLibraryFile.h"
> > > -#include "lld/Core/DefinedAtom.h"
> > > -#include "lld/Core/File.h"
> > > -#include "lld/Core/LLVM.h"
> > > -#include "lld/Core/Reference.h"
> > > -#include "lld/Core/SharedLibraryFile.h"
> > > -
> > > -namespace lld {
> > > -
> > > -
> > > -/// \brief Parse the input file to lld::File.
> > > -std::error_code MachOFileNode::parse(const LinkingContext &ctx,
> > > - raw_ostream &diagnostics) {
> > > - ErrorOr<StringRef> filePath = getPath(ctx);
> > > - if (std::error_code ec = filePath.getError())
> > > - return ec;
> > > - ErrorOr<std::unique_ptr<MemoryBuffer>> mbOrErr =
> > > - MemoryBuffer::getFileOrSTDIN(*filePath);
> > > - if (std::error_code ec = mbOrErr.getError())
> > > - return ec;
> > > - std::unique_ptr<MemoryBuffer> mb = std::move(mbOrErr.get());
> > > -
> > > - _context.addInputFileDependency(*filePath);
> > > - if (ctx.logInputFiles())
> > > - diagnostics << *filePath << "\n";
> > > -
> > > - narrowFatBuffer(mb, *filePath);
> > > -
> > > - std::vector<std::unique_ptr<File>> parsedFiles;
> > > - if (std::error_code ec = ctx.registry().parseFile(std::move(mb), parsedFiles))
> > > - return ec;
> > > - for (std::unique_ptr<File> &pf : parsedFiles) {
> > > - // If file is a dylib, inform LinkingContext about it.
> > > - if (SharedLibraryFile *shl = dyn_cast<SharedLibraryFile>(pf.get())) {
> > > - _context.registerDylib(reinterpret_cast<mach_o::MachODylibFile*>(shl),
> > > - _upwardDylib);
> > > - }
> > > - // If file is an archive and -all_load, then add all members.
> > > - if (ArchiveLibraryFile *archive = dyn_cast<ArchiveLibraryFile>(pf.get())) {
> > > - if (_isWholeArchive) {
> > > - // Have this node own the FileArchive object.
> > > - _archiveFile.reset(archive);
> > > - pf.release();
> > > - // Add all members to _files vector
> > > - return archive->parseAllMembers(_files);
> > > - }
> > > - }
> > > - _files.push_back(std::move(pf));
> > > - }
> > > - return std::error_code();
> > > -}
> > > -
> > > -
> > > -/// If buffer contains a fat file, find required arch in fat buffer and
> > > -/// switch buffer to point to just that required slice.
> > > -void MachOFileNode::narrowFatBuffer(std::unique_ptr<MemoryBuffer> &mb,
> > > - StringRef filePath) {
> > > - // Check if buffer is a "fat" file that contains needed arch.
> > > - uint32_t offset;
> > > - uint32_t size;
> > > - if (!_context.sliceFromFatFile(*mb, offset, size)) {
> > > - return;
> > > - }
> > > - // Create new buffer containing just the needed slice.
> > > - auto subuf = MemoryBuffer::getFileSlice(filePath, size, offset);
> > > - if (subuf.getError())
> > > - return;
> > > - // The assignment to mb will release previous buffer.
> > > - mb = std::move(subuf.get());
> > > -}
> > > -
> > > -
> > > -} // end namesapce lld
> > >
> > > Modified: lld/trunk/lib/Driver/DarwinLdDriver.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Driver/DarwinLdDriver.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/Driver/DarwinLdDriver.cpp (original)
> > > +++ lld/trunk/lib/Driver/DarwinLdDriver.cpp Mon Jan 12 22:33:07 2015
> > > @@ -13,8 +13,11 @@
> > > ///
> > > //===----------------------------------------------------------------------===//
> > >
> > > +#include "lld/Core/File.h"
> > > +#include "lld/Core/ArchiveLibraryFile.h"
> > > +#include "lld/Core/SharedLibraryFile.h"
> > > #include "lld/Driver/Driver.h"
> > > -#include "lld/Driver/DarwinInputGraph.h"
> > > +#include "lld/Driver/WrapperInputGraph.h"
> > > #include "lld/ReaderWriter/MachOLinkingContext.h"
> > > #include "llvm/ADT/ArrayRef.h"
> > > #include "llvm/ADT/STLExtras.h"
> > > @@ -22,6 +25,7 @@
> > > #include "llvm/Option/Arg.h"
> > > #include "llvm/Option/Option.h"
> > > #include "llvm/Support/CommandLine.h"
> > > +#include "llvm/Support/Debug.h"
> > > #include "llvm/Support/FileSystem.h"
> > > #include "llvm/Support/Format.h"
> > > #include "llvm/Support/Host.h"
> > > @@ -68,6 +72,50 @@ public:
> > > DarwinLdOptTable() : OptTable(infoTable, llvm::array_lengthof(infoTable)){}
> > > };
> > >
> > > +std::vector<std::unique_ptr<File>>
> > > +loadFiles(MachOLinkingContext &ctx, StringRef path,
> > > + raw_ostream &diag, bool wholeArchive, bool upwardDylib) {
> > > + ErrorOr<std::unique_ptr<MemoryBuffer>> mbOrErr =
> > > + MemoryBuffer::getFileOrSTDIN(path);
> > > + if (std::error_code ec = mbOrErr.getError())
> > > + return makeErrorFile(path, ec);
> > > + std::unique_ptr<MemoryBuffer> mb = std::move(mbOrErr.get());
> > > +
> > > + ctx.addInputFileDependency(path);
> > > + if (ctx.logInputFiles())
> > > + diag << path << "\n";
> > > +
> > > + // If buffer contains a fat file, find required arch in fat buffer
> > > + // and switch buffer to point to just that required slice.
> > > + // Check if buffer is a "fat" file that contains needed arch.
> > > + uint32_t offset;
> > > + uint32_t size;
> > > + if (ctx.sliceFromFatFile(*mb, offset, size)) {
> > > + // Create new buffer containing just the needed slice.
> > > + auto subuf = MemoryBuffer::getFileSlice(path, size, offset);
> > > + if (std::error_code ec = subuf.getError())
> > > + return makeErrorFile(path, ec);
> > > + // The assignment to mb will release previous buffer.
> > > + mb = std::move(subuf.get());
> > > + }
> > > +
> > > + std::vector<std::unique_ptr<File>> files;
> > > + if (std::error_code ec = ctx.registry().loadFile(std::move(mb), files))
> > > + return makeErrorFile(path, ec);
> > > + for (std::unique_ptr<File> &pf : files) {
> > > + // If file is a dylib, inform LinkingContext about it.
> > > + if (SharedLibraryFile *shl = dyn_cast<SharedLibraryFile>(pf.get())) {
> > > + if (std::error_code ec = shl->parse())
> > > + return makeErrorFile(path, ec);
> > > + ctx.registerDylib(reinterpret_cast<mach_o::MachODylibFile*>(shl),
> > > + upwardDylib);
> > > + }
> > > + }
> > > + if (wholeArchive)
> > > + return parseMemberFiles(files);
> > > + return files;
> > > +}
> > > +
> > > } // anonymous namespace
> > >
> > > // Test may be running on Windows. Canonicalize the path
> > > @@ -85,13 +133,12 @@ static std::string canonicalizePath(Stri
> > >
> > > static void addFile(StringRef path, std::unique_ptr<InputGraph> &inputGraph,
> > > MachOLinkingContext &ctx, bool loadWholeArchive,
> > > - bool upwardDylib) {
> > > - auto node = llvm::make_unique<MachOFileNode>(path, ctx);
> > > - if (loadWholeArchive)
> > > - node->setLoadWholeArchive();
> > > - if (upwardDylib)
> > > - node->setUpwardDylib();
> > > - inputGraph->addInputElement(std::move(node));
> > > + bool upwardDylib, raw_ostream &diag) {
> > > + std::vector<std::unique_ptr<File>> files =
> > > + loadFiles(ctx, path, diag, loadWholeArchive, upwardDylib);
> > > + for (std::unique_ptr<File> &file : files)
> > > + inputGraph->addInputElement(
> > > + llvm::make_unique<WrapperNode>(std::move(file)));
> > > }
> > >
> > > // Export lists are one symbol per line. Blank lines are ignored.
> > > @@ -184,10 +231,10 @@ static std::error_code parseOrderFile(St
> > > // In this variant, the path is to a text file which contains a partial path
> > > // per line. The <dir> prefix is prepended to each partial path.
> > > //
> > > -static std::error_code parseFileList(StringRef fileListPath,
> > > - std::unique_ptr<InputGraph> &inputGraph,
> > > - MachOLinkingContext &ctx, bool forceLoad,
> > > - raw_ostream &diagnostics) {
> > > +static std::error_code loadFileList(StringRef fileListPath,
> > > + std::unique_ptr<InputGraph> &inputGraph,
> > > + MachOLinkingContext &ctx, bool forceLoad,
> > > + raw_ostream &diagnostics) {
> > > // If there is a comma, split off <dir>.
> > > std::pair<StringRef, StringRef> opt = fileListPath.split(',');
> > > StringRef filePath = opt.first;
> > > @@ -222,7 +269,7 @@ static std::error_code parseFileList(Str
> > > if (ctx.testingFileUsage()) {
> > > diagnostics << "Found filelist entry " << canonicalizePath(path) << '\n';
> > > }
> > > - addFile(path, inputGraph, ctx, forceLoad, false);
> > > + addFile(path, inputGraph, ctx, forceLoad, false, diagnostics);
> > > buffer = lineAndRest.second;
> > > }
> > > return std::error_code();
> > > @@ -244,13 +291,6 @@ bool DarwinLdDriver::linkMachO(int argc,
> > > return false;
> > > if (ctx.doNothing())
> > > return true;
> > > -
> > > - // Register possible input file parsers.
> > > - ctx.registry().addSupportMachOObjects(ctx);
> > > - ctx.registry().addSupportArchives(ctx.logInputFiles());
> > > - ctx.registry().addSupportNativeObjects();
> > > - ctx.registry().addSupportYamlFiles();
> > > -
> > > return link(ctx, diagnostics);
> > > }
> > >
> > > @@ -310,15 +350,15 @@ bool DarwinLdDriver::parse(int argc, con
> > > }
> > > }
> > > // If no -arch specified, scan input files to find first non-fat .o file.
> > > - if ((arch == MachOLinkingContext::arch_unknown)
> > > - && !parsedArgs->getLastArg(OPT_test_file_usage)) {
> > > + if (arch == MachOLinkingContext::arch_unknown) {
> > > for (auto &inFile: parsedArgs->filtered(OPT_INPUT)) {
> > > // This is expensive because it opens and maps the file. But that is
> > > // ok because no -arch is rare.
> > > if (MachOLinkingContext::isThinObjectFile(inFile->getValue(), arch))
> > > break;
> > > }
> > > - if (arch == MachOLinkingContext::arch_unknown) {
> > > + if (arch == MachOLinkingContext::arch_unknown
> > > + && !parsedArgs->getLastArg(OPT_test_file_usage)) {
> > > // If no -arch and no options at all, print usage message.
> > > if (parsedArgs->size() == 0)
> > > table.PrintHelp(llvm::outs(), argv[0], "LLVM Linker", false);
> > > @@ -521,6 +561,13 @@ bool DarwinLdDriver::parse(int argc, con
> > > }
> > > }
> > >
> > > + // Register possible input file parsers.
> > > + if (!ctx.doNothing()) {
> > > + ctx.registry().addSupportMachOObjects(ctx);
> > > + ctx.registry().addSupportArchives(ctx.logInputFiles());
> > > + ctx.registry().addSupportNativeObjects();
> > > + ctx.registry().addSupportYamlFiles();
> > > + }
> > > std::unique_ptr<InputGraph> inputGraph(new InputGraph());
> > >
> > > // Now construct the set of library search directories, following ld64's
> > > @@ -745,13 +792,13 @@ bool DarwinLdDriver::parse(int argc, con
> > > default:
> > > continue;
> > > case OPT_INPUT:
> > > - addFile(arg->getValue(), inputGraph, ctx, globalWholeArchive, false);
> > > + addFile(arg->getValue(), inputGraph, ctx, globalWholeArchive, false, diagnostics);
> > > break;
> > > case OPT_upward_library:
> > > - addFile(arg->getValue(), inputGraph, ctx, false, true);
> > > + addFile(arg->getValue(), inputGraph, ctx, false, true, diagnostics);
> > > break;
> > > case OPT_force_load:
> > > - addFile(arg->getValue(), inputGraph, ctx, true, false);
> > > + addFile(arg->getValue(), inputGraph, ctx, true, false, diagnostics);
> > > break;
> > > case OPT_l:
> > > case OPT_upward_l:
> > > @@ -765,7 +812,7 @@ bool DarwinLdDriver::parse(int argc, con
> > > diagnostics << "Found " << (upward ? "upward " : " ") << "library "
> > > << canonicalizePath(resolvedPath.get()) << '\n';
> > > }
> > > - addFile(resolvedPath.get(), inputGraph, ctx, globalWholeArchive, upward);
> > > + addFile(resolvedPath.get(), inputGraph, ctx, globalWholeArchive, upward, diagnostics);
> > > break;
> > > case OPT_framework:
> > > case OPT_upward_framework:
> > > @@ -779,12 +826,12 @@ bool DarwinLdDriver::parse(int argc, con
> > > diagnostics << "Found " << (upward ? "upward " : " ") << "framework "
> > > << canonicalizePath(resolvedPath.get()) << '\n';
> > > }
> > > - addFile(resolvedPath.get(), inputGraph, ctx, globalWholeArchive, upward);
> > > + addFile(resolvedPath.get(), inputGraph, ctx, globalWholeArchive, upward, diagnostics);
> > > break;
> > > case OPT_filelist:
> > > - if (std::error_code ec = parseFileList(arg->getValue(), inputGraph,
> > > - ctx, globalWholeArchive,
> > > - diagnostics)) {
> > > + if (std::error_code ec = loadFileList(arg->getValue(), inputGraph,
> > > + ctx, globalWholeArchive,
> > > + diagnostics)) {
> > > diagnostics << "error: " << ec.message()
> > > << ", processing '-filelist " << arg->getValue()
> > > << "'\n";
> > >
> > > Modified: lld/trunk/lib/Driver/Driver.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Driver/Driver.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/Driver/Driver.cpp (original)
> > > +++ lld/trunk/lib/Driver/Driver.cpp Mon Jan 12 22:33:07 2015
> > > @@ -49,13 +49,13 @@ FileVector parseMemberFiles(FileVector &
> > > return members;
> > > }
> > >
> > > -FileVector parseFile(LinkingContext &ctx, StringRef path, bool wholeArchive) {
> > > +FileVector loadFile(LinkingContext &ctx, StringRef path, bool wholeArchive) {
> > > ErrorOr<std::unique_ptr<MemoryBuffer>> mb
> > > = MemoryBuffer::getFileOrSTDIN(path);
> > > if (std::error_code ec = mb.getError())
> > > return makeErrorFile(path, ec);
> > > std::vector<std::unique_ptr<File>> files;
> > > - if (std::error_code ec = ctx.registry().parseFile(std::move(mb.get()), files))
> > > + if (std::error_code ec = ctx.registry().loadFile(std::move(mb.get()), files))
> > > return makeErrorFile(path, ec);
> > > if (wholeArchive)
> > > return parseMemberFiles(files);
> > >
> > > Modified: lld/trunk/lib/Driver/GnuLdDriver.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Driver/GnuLdDriver.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/Driver/GnuLdDriver.cpp (original)
> > > +++ lld/trunk/lib/Driver/GnuLdDriver.cpp Mon Jan 12 22:33:07 2015
> > > @@ -14,7 +14,8 @@
> > > //===----------------------------------------------------------------------===//
> > >
> > > #include "lld/Driver/Driver.h"
> > > -#include "lld/Driver/GnuLdInputGraph.h"
> > > +#include "lld/Driver/WrapperInputGraph.h"
> > > +#include "lld/ReaderWriter/ELFLinkingContext.h"
> > > #include "lld/ReaderWriter/LinkerScript.h"
> > > #include "llvm/ADT/ArrayRef.h"
> > > #include "llvm/ADT/Optional.h"
> > > @@ -172,16 +173,6 @@ bool GnuLdDriver::linkELF(int argc, cons
> > > return false;
> > > if (!options)
> > > return true;
> > > -
> > > - // Register possible input file parsers.
> > > - options->registry().addSupportELFObjects(options->mergeCommonStrings(),
> > > - options->targetHandler());
> > > - options->registry().addSupportArchives(options->logInputFiles());
> > > - options->registry().addSupportYamlFiles();
> > > - options->registry().addSupportNativeObjects();
> > > - if (options->allowLinkWithDynamicLibraries())
> > > - options->registry().addSupportELFDynamicSharedObjects(
> > > - options->useShlibUndefines(), options->targetHandler());
> > > return link(*options, diagnostics);
> > > }
> > >
> > > @@ -218,6 +209,23 @@ static bool isLinkerScript(StringRef pat
> > > return magic == llvm::sys::fs::file_magic::unknown;
> > > }
> > >
> > > +static ErrorOr<StringRef>
> > > +findFile(ELFLinkingContext &ctx, StringRef path, bool dashL) {
> > > + // If the path was referred to by using a -l argument, let's search
> > > + // for the file in the search path.
> > > + if (dashL) {
> > > + ErrorOr<StringRef> pathOrErr = ctx.searchLibrary(path);
> > > + if (std::error_code ec = pathOrErr.getError())
> > > + return make_dynamic_error_code(
> > > + Twine("Unable to find library -l") + path + ": " + ec.message());
> > > + path = pathOrErr->str();
> > > + }
> > > + if (!llvm::sys::fs::exists(path))
> > > + return make_dynamic_error_code(
> > > + Twine("lld: cannot find file ") + path);
> > > + return path;
> > > +}
> > > +
> > > static bool isPathUnderSysroot(StringRef sysroot, StringRef path) {
> > > if (sysroot.empty())
> > > return false;
> > > @@ -234,8 +242,6 @@ evaluateLinkerScript(ELFLinkingContext &
> > > MemoryBuffer::getFileOrSTDIN(path);
> > > if (std::error_code ec = mb.getError())
> > > return ec;
> > > - if (ctx.logInputFiles())
> > > - diag << path << "\n";
> > > auto lexer = llvm::make_unique<script::Lexer>(std::move(mb.get()));
> > > auto parser = llvm::make_unique<script::Parser>(*lexer);
> > > script::LinkerScript *script = parser->parse();
> > > @@ -253,13 +259,26 @@ evaluateLinkerScript(ELFLinkingContext &
> > > int numfiles = 0;
> > > for (const script::Path &path : group->getPaths()) {
> > > // TODO : Propagate Set WholeArchive/dashlPrefix
> > > - ELFFileNode::Attributes attr;
> > > + ELFLinkingContext::Attributes attr;
> > > attr.setSysRooted(sysroot);
> > > attr.setAsNeeded(path._asNeeded);
> > > attr.setDashlPrefix(path._isDashlPrefix);
> > > - ++numfiles;
> > > - inputGraph->addInputElement(llvm::make_unique<ELFFileNode>(
> > > - ctx, ctx.allocateString(path._path), attr));
> > > +
> > > + ErrorOr<StringRef> pathOrErr = path._isDashlPrefix
> > > + ? ctx.searchLibrary(path._path) : ctx.searchFile(path._path, sysroot);
> > > + if (std::error_code ec = pathOrErr.getError())
> > > + return make_dynamic_error_code(
> > > + Twine("Unable to find file ") + path._path + ": " + ec.message());
> > > +
> > > + std::vector<std::unique_ptr<File>> files
> > > + = loadFile(ctx, pathOrErr.get(), false);
> > > + for (std::unique_ptr<File> &file : files) {
> > > + if (ctx.logInputFiles())
> > > + diag << file->path() << "\n";
> > > + inputGraph->addInputElement(
> > > + std::unique_ptr<InputElement>(new WrapperNode(std::move(file))));
> > > + ++numfiles;
> > > + }
> > > }
> > > inputGraph->addInputElement(llvm::make_unique<GroupEnd>(numfiles));
> > > }
> > > @@ -341,7 +360,7 @@ bool GnuLdDriver::parse(int argc, const
> > > std::stack<int> groupStack;
> > > int numfiles = 0;
> > >
> > > - ELFFileNode::Attributes attributes;
> > > + ELFLinkingContext::Attributes attributes;
> > >
> > > bool _outputOptionSet = false;
> > >
> > > @@ -379,7 +398,6 @@ bool GnuLdDriver::parse(int argc, const
> > > ctx->setPrintRemainingUndefines(false);
> > > ctx->setAllowRemainingUndefines(true);
> > > break;
> > > -
> > > case OPT_static:
> > > ctx->setOutputELFType(llvm::ELF::ET_EXEC);
> > > ctx->setIsStaticExecutable(true);
> > > @@ -415,6 +433,37 @@ bool GnuLdDriver::parse(int argc, const
> > > }
> > > }
> > >
> > > + for (auto inputArg : *parsedArgs) {
> > > + switch (inputArg->getOption().getID()) {
> > > + case OPT_merge_strings:
> > > + ctx->setMergeCommonStrings(true);
> > > + break;
> > > + case OPT_t:
> > > + ctx->setLogInputFiles(true);
> > > + break;
> > > + case OPT_use_shlib_undefs:
> > > + ctx->setUseShlibUndefines(true);
> > > + break;
> > > + case OPT_no_allow_shlib_undefs:
> > > + ctx->setAllowShlibUndefines(false);
> > > + break;
> > > + case OPT_allow_shlib_undefs:
> > > + ctx->setAllowShlibUndefines(true);
> > > + break;
> > > + }
> > > + }
> > > +
> > > + // Register possible input file parsers.
> > > + ctx->registry().addSupportELFObjects(
> > > + ctx->mergeCommonStrings(),
> > > + ctx->targetHandler());
> > > + ctx->registry().addSupportArchives(ctx->logInputFiles());
> > > + ctx->registry().addSupportYamlFiles();
> > > + ctx->registry().addSupportNativeObjects();
> > > + if (ctx->allowLinkWithDynamicLibraries())
> > > + ctx->registry().addSupportELFDynamicSharedObjects(
> > > + ctx->useShlibUndefines(), ctx->targetHandler());
> > > +
> > > // Process all the arguments and create Input Elements
> > > for (auto inputArg : *parsedArgs) {
> > > switch (inputArg->getOption().getID()) {
> > > @@ -438,26 +487,6 @@ bool GnuLdDriver::parse(int argc, const
> > > ctx->setExportDynamic(true);
> > > break;
> > >
> > > - case OPT_merge_strings:
> > > - ctx->setMergeCommonStrings(true);
> > > - break;
> > > -
> > > - case OPT_t:
> > > - ctx->setLogInputFiles(true);
> > > - break;
> > > -
> > > - case OPT_no_allow_shlib_undefs:
> > > - ctx->setAllowShlibUndefines(false);
> > > - break;
> > > -
> > > - case OPT_allow_shlib_undefs:
> > > - ctx->setAllowShlibUndefines(true);
> > > - break;
> > > -
> > > - case OPT_use_shlib_undefs:
> > > - ctx->setUseShlibUndefines(true);
> > > - break;
> > > -
> > > case OPT_allow_multiple_definition:
> > > ctx->setAllowDuplicates(true);
> > > break;
> > > @@ -560,25 +589,19 @@ bool GnuLdDriver::parse(int argc, const
> > > bool dashL = (inputArg->getOption().getID() == OPT_l);
> > > attributes.setDashlPrefix(dashL);
> > > StringRef path = inputArg->getValue();
> > > - std::string realpath = path;
> > >
> > > - // If the path was referred to by using a -l argument, let's search
> > > - // for the file in the search path.
> > > - if (dashL) {
> > > - ErrorOr<StringRef> pathOrErr = ctx->searchLibrary(path);
> > > - if (!pathOrErr) {
> > > - diagnostics << " Unable to find library -l" << path << "\n";
> > > - return false;
> > > - }
> > > - realpath = pathOrErr->str();
> > > - }
> > > - if (!llvm::sys::fs::exists(realpath)) {
> > > - diagnostics << "lld: cannot find file " << path << "\n";
> > > + ErrorOr<StringRef> pathOrErr = findFile(*ctx, path, dashL);
> > > + if (std::error_code ec = pathOrErr.getError()) {
> > > + diagnostics << ec.message() << "\n";
> > > return false;
> > > }
> > > + std::string realpath = pathOrErr.get();
> > > +
> > > bool isScript =
> > > (!path.endswith(".objtxt") && isLinkerScript(realpath, diagnostics));
> > > if (isScript) {
> > > + if (ctx->logInputFiles())
> > > + diagnostics << path << "\n";
> > > std::error_code ec = evaluateLinkerScript(
> > > *ctx, inputGraph.get(), realpath, diagnostics);
> > > if (ec) {
> > > @@ -588,9 +611,15 @@ bool GnuLdDriver::parse(int argc, const
> > > }
> > > break;
> > > }
> > > - ++numfiles;
> > > - inputGraph->addInputElement(
> > > - llvm::make_unique<ELFFileNode>(*ctx, path, attributes));
> > > + std::vector<std::unique_ptr<File>> files
> > > + = loadFile(*ctx, realpath, attributes._isWholeArchive);
> > > + for (std::unique_ptr<File> &file : files) {
> > > + if (ctx->logInputFiles())
> > > + diagnostics << file->path() << "\n";
> > > + inputGraph->addInputElement(
> > > + std::unique_ptr<InputElement>(new WrapperNode(std::move(file))));
> > > + }
> > > + numfiles += files.size();
> > > break;
> > > }
> > >
> > >
> > > Removed: lld/trunk/lib/Driver/GnuLdInputGraph.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Driver/GnuLdInputGraph.cpp?rev=225763&view=auto
> > > ==============================================================================
> > > --- lld/trunk/lib/Driver/GnuLdInputGraph.cpp (original)
> > > +++ lld/trunk/lib/Driver/GnuLdInputGraph.cpp (removed)
> > > @@ -1,65 +0,0 @@
> > > -//===- lib/Driver/GnuLdInputGraph.cpp -------------------------------------===//
> > > -//
> > > -// The LLVM Linker
> > > -//
> > > -// This file is distributed under the University of Illinois Open Source
> > > -// License. See LICENSE.TXT for details.
> > > -//
> > > -//===----------------------------------------------------------------------===//
> > > -
> > > -#include "lld/Driver/GnuLdInputGraph.h"
> > > -#include "lld/ReaderWriter/LinkerScript.h"
> > > -#include "llvm/Support/Errc.h"
> > > -#include "llvm/Support/FileSystem.h"
> > > -#include "llvm/Support/Path.h"
> > > -
> > > -using namespace lld;
> > > -
> > > -llvm::ErrorOr<StringRef> ELFFileNode::getPath(const LinkingContext &) const {
> > > - if (_attributes._isDashlPrefix)
> > > - return _elfLinkingContext.searchLibrary(_path);
> > > - return _elfLinkingContext.searchFile(_path, _attributes._isSysRooted);
> > > -}
> > > -
> > > -std::string ELFFileNode::errStr(std::error_code errc) {
> > > - if (errc == llvm::errc::no_such_file_or_directory) {
> > > - if (_attributes._isDashlPrefix)
> > > - return (Twine("Unable to find library -l") + _path).str();
> > > - return (Twine("Unable to find file ") + _path).str();
> > > - }
> > > - return FileNode::errStr(errc);
> > > -}
> > > -
> > > -/// \brief Parse the input file to lld::File.
> > > -std::error_code ELFFileNode::parse(const LinkingContext &ctx,
> > > - raw_ostream &diagnostics) {
> > > - ErrorOr<StringRef> filePath = getPath(ctx);
> > > - if (std::error_code ec = filePath.getError())
> > > - return ec;
> > > - ErrorOr<std::unique_ptr<MemoryBuffer>> mb =
> > > - MemoryBuffer::getFileOrSTDIN(*filePath);
> > > - if (std::error_code ec = mb.getError())
> > > - return ec;
> > > - if (ctx.logInputFiles())
> > > - diagnostics << *filePath << "\n";
> > > -
> > > - if (_attributes._isWholeArchive) {
> > > - std::vector<std::unique_ptr<File>> parsedFiles;
> > > - if (std::error_code ec = ctx.registry().parseFile(
> > > - std::move(mb.get()), parsedFiles))
> > > - return ec;
> > > - assert(parsedFiles.size() == 1);
> > > - std::unique_ptr<File> f(parsedFiles[0].release());
> > > - if (const auto *archive = dyn_cast<ArchiveLibraryFile>(f.get())) {
> > > - // Have this node own the FileArchive object.
> > > - _archiveFile.reset(archive);
> > > - f.release();
> > > - // Add all members to _files vector
> > > - return archive->parseAllMembers(_files);
> > > - }
> > > - // if --whole-archive is around non-archive, just use it as normal.
> > > - _files.push_back(std::move(f));
> > > - return std::error_code();
> > > - }
> > > - return ctx.registry().parseFile(std::move(mb.get()), _files);
> > > -}
> > >
> > > Modified: lld/trunk/lib/Driver/WinLinkDriver.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Driver/WinLinkDriver.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/Driver/WinLinkDriver.cpp (original)
> > > +++ lld/trunk/lib/Driver/WinLinkDriver.cpp Mon Jan 12 22:33:07 2015
> > > @@ -14,8 +14,8 @@
> > > //===----------------------------------------------------------------------===//
> > >
> > > #include "lld/Driver/Driver.h"
> > > -#include "lld/Driver/WinLinkInputGraph.h"
> > > #include "lld/Driver/WinLinkModuleDef.h"
> > > +#include "lld/Driver/WrapperInputGraph.h"
> > > #include "lld/ReaderWriter/PECOFFLinkingContext.h"
> > > #include "llvm/ADT/ArrayRef.h"
> > > #include "llvm/ADT/Optional.h"
> > > @@ -800,13 +800,11 @@ parseArgs(int argc, const char **argv, P
> > >
> > > // Returns true if the given file node has already been added to the input
> > > // graph.
> > > -static bool hasLibrary(const PECOFFLinkingContext &ctx, FileNode *fileNode) {
> > > - ErrorOr<StringRef> path = fileNode->getPath(ctx);
> > > - if (!path)
> > > - return false;
> > > +static bool hasLibrary(const PECOFFLinkingContext &ctx, File *file) {
> > > + StringRef path = file->path();
> > > for (std::unique_ptr<InputElement> &p : ctx.getInputGraph().inputElements())
> > > if (auto *f = dyn_cast<FileNode>(p.get()))
> > > - if (*path == *f->getPath(ctx))
> > > + if (*f->getPath(ctx) == path)
> > > return true;
> > > return false;
> > > }
> > > @@ -838,6 +836,16 @@ static bool maybeRunLibCommand(int argc,
> > > return true;
> > > }
> > >
> > > +/// \brief Parse the input file to lld::File.
> > > +void addFiles(PECOFFLinkingContext &ctx, StringRef path, raw_ostream &diag,
> > > + std::vector<std::unique_ptr<File>> &files) {
> > > + for (std::unique_ptr<File> &file : loadFile(ctx, path, false)) {
> > > + if (ctx.logInputFiles())
> > > + diag << file->path() << "\n";
> > > + files.push_back(std::move(file));
> > > + }
> > > +}
> > > +
> > > //
> > > // Main driver
> > > //
> > > @@ -847,6 +855,12 @@ bool WinLinkDriver::linkPECOFF(int argc,
> > > return true;
> > >
> > > PECOFFLinkingContext ctx;
> > > + ctx.registry().addSupportCOFFObjects(ctx);
> > > + ctx.registry().addSupportCOFFImportLibraries(ctx);
> > > + ctx.registry().addSupportArchives(ctx.logInputFiles());
> > > + ctx.registry().addSupportNativeObjects();
> > > + ctx.registry().addSupportYamlFiles();
> > > +
> > > std::vector<const char *> newargv = processLinkEnv(ctx, argc, argv);
> > > processLibEnv(ctx);
> > > if (!parse(newargv.size() - 1, &newargv[0], ctx, diag))
> > > @@ -857,13 +871,6 @@ bool WinLinkDriver::linkPECOFF(int argc,
> > > if (!createSideBySideManifestFile(ctx, diag))
> > > return false;
> > >
> > > - // Register possible input file parsers.
> > > - ctx.registry().addSupportCOFFObjects(ctx);
> > > - ctx.registry().addSupportCOFFImportLibraries(ctx);
> > > - ctx.registry().addSupportArchives(ctx.logInputFiles());
> > > - ctx.registry().addSupportNativeObjects();
> > > - ctx.registry().addSupportYamlFiles();
> > > -
> > > return link(ctx, diag);
> > > }
> > >
> > > @@ -884,8 +891,8 @@ bool WinLinkDriver::parse(int argc, cons
> > > return false;
> > >
> > > // The list of input files.
> > > - std::vector<std::unique_ptr<FileNode> > files;
> > > - std::vector<std::unique_ptr<FileNode> > libraries;
> > > + std::vector<std::unique_ptr<File>> files;
> > > + std::vector<std::unique_ptr<File>> libraries;
> > >
> > > // Handle /help
> > > if (parsedArgs->getLastArg(OPT_help)) {
> > > @@ -1363,11 +1370,9 @@ bool WinLinkDriver::parse(int argc, cons
> > > for (StringRef path : inputFiles) {
> > > path = ctx.allocate(path);
> > > if (isLibraryFile(path)) {
> > > - libraries.push_back(std::unique_ptr<FileNode>(
> > > - new PECOFFFileNode(ctx, getLibraryPath(ctx, path))));
> > > + addFiles(ctx, getLibraryPath(ctx, path), diag, libraries);
> > > } else {
> > > - files.push_back(std::unique_ptr<FileNode>(
> > > - new PECOFFFileNode(ctx, getObjectPath(ctx, path))));
> > > + addFiles(ctx, getObjectPath(ctx, path), diag, files);
> > > }
> > > }
> > >
> > > @@ -1389,8 +1394,7 @@ bool WinLinkDriver::parse(int argc, cons
> > > if (!ctx.getNoDefaultLibAll())
> > > for (const StringRef path : defaultLibs)
> > > if (!ctx.hasNoDefaultLib(path))
> > > - libraries.push_back(std::unique_ptr<FileNode>(
> > > - new PECOFFFileNode(ctx, getLibraryPath(ctx, path.lower()))));
> > > + addFiles(ctx, getLibraryPath(ctx, path.lower()), diag, libraries);
> > >
> > > if (files.empty() && !isReadingDirectiveSection) {
> > > diag << "No input files\n";
> > > @@ -1401,18 +1405,19 @@ bool WinLinkDriver::parse(int argc, cons
> > > // constructed by replacing an extension of the first input file
> > > // with ".exe".
> > > if (ctx.outputPath().empty()) {
> > > - StringRef path = *cast<FileNode>(&*files[0])->getPath(ctx);
> > > + StringRef path = files[0]->path();
> > > ctx.setOutputPath(replaceExtension(ctx, path, ".exe"));
> > > }
> > >
> > > // Add the input files to the input graph.
> > > if (!ctx.hasInputGraph())
> > > ctx.setInputGraph(std::unique_ptr<InputGraph>(new InputGraph()));
> > > - for (std::unique_ptr<FileNode> &file : files) {
> > > + for (std::unique_ptr<File> &file : files) {
> > > if (isReadingDirectiveSection)
> > > - if (file->parse(ctx, diag))
> > > + if (file->parse())
> > > return false;
> > > - ctx.getInputGraph().addInputElement(std::move(file));
> > > + ctx.getInputGraph().addInputElement(
> > > + std::unique_ptr<InputElement>(new WrapperNode(std::move(file))));
> > > }
> > >
> > > // Add the library group to the input graph.
> > > @@ -1427,12 +1432,13 @@ bool WinLinkDriver::parse(int argc, cons
> > > }
> > >
> > > // Add the library files to the library group.
> > > - for (std::unique_ptr<FileNode> &lib : libraries) {
> > > - if (!hasLibrary(ctx, lib.get())) {
> > > + for (std::unique_ptr<File> &file : libraries) {
> > > + if (!hasLibrary(ctx, file.get())) {
> > > if (isReadingDirectiveSection)
> > > - if (lib->parse(ctx, diag))
> > > + if (file->parse())
> > > return false;
> > > - ctx.addLibraryFile(std::move(lib));
> > > + ctx.addLibraryFile(
> > > + std::unique_ptr<FileNode>(new WrapperNode(std::move(file))));
> > > }
> > > }
> > >
> > >
> > > Removed: lld/trunk/lib/Driver/WinLinkInputGraph.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Driver/WinLinkInputGraph.cpp?rev=225763&view=auto
> > > ==============================================================================
> > > --- lld/trunk/lib/Driver/WinLinkInputGraph.cpp (original)
> > > +++ lld/trunk/lib/Driver/WinLinkInputGraph.cpp (removed)
> > > @@ -1,39 +0,0 @@
> > > -//===- lib/Driver/WinLinkInputGraph.cpp -----------------------------------===//
> > > -//
> > > -// The LLVM Linker
> > > -//
> > > -// This file is distributed under the University of Illinois Open Source
> > > -// License. See LICENSE.TXT for details.
> > > -//
> > > -//===----------------------------------------------------------------------===//
> > > -
> > > -#include "lld/Driver/WinLinkInputGraph.h"
> > > -
> > > -namespace lld {
> > > -
> > > -/// \brief Parse the input file to lld::File.
> > > -std::error_code PECOFFFileNode::parse(const LinkingContext &ctx,
> > > - raw_ostream &diagnostics) {
> > > - if (_parsed)
> > > - return std::error_code();
> > > - _parsed = true;
> > > - ErrorOr<StringRef> filePath = getPath(ctx);
> > > - if (std::error_code ec = filePath.getError()) {
> > > - diagnostics << "File not found: " << _path << "\n";
> > > - return ec;
> > > - }
> > > -
> > > - ErrorOr<std::unique_ptr<MemoryBuffer>> mb =
> > > - MemoryBuffer::getFileOrSTDIN(*filePath);
> > > - if (std::error_code ec = mb.getError()) {
> > > - diagnostics << "Cannot open file: " << *filePath << "\n";
> > > - return ec;
> > > - }
> > > -
> > > - if (ctx.logInputFiles())
> > > - diagnostics << *filePath << "\n";
> > > -
> > > - return ctx.registry().parseFile(std::move(mb.get()), _files);
> > > -}
> > > -
> > > -} // end anonymous namespace
> > >
> > > Modified: lld/trunk/lib/Passes/RoundTripNativePass.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Passes/RoundTripNativePass.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/Passes/RoundTripNativePass.cpp (original)
> > > +++ lld/trunk/lib/Passes/RoundTripNativePass.cpp Mon Jan 12 22:33:07 2015
> > > @@ -40,13 +40,15 @@ void RoundTripNativePass::perform(std::u
> > > if (!mb)
> > > return;
> > >
> > > - std::error_code ec = _context.registry().parseFile(
> > > + std::error_code ec = _context.registry().loadFile(
> > > std::move(mb.get()), _nativeFile);
> > > if (ec) {
> > > // Note: we need a way for Passes to report errors.
> > > llvm_unreachable("native reader not registered or read error");
> > > }
> > > File *objFile = _nativeFile[0].get();
> > > + if (objFile->parse())
> > > + llvm_unreachable("native reader parse error");
> > > mergedFile.reset(new SimpleFileWrapper(_context, *objFile));
> > >
> > > llvm::sys::fs::remove(tmpNativeFile.str());
> > >
> > > Modified: lld/trunk/lib/Passes/RoundTripYAMLPass.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/Passes/RoundTripYAMLPass.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/Passes/RoundTripYAMLPass.cpp (original)
> > > +++ lld/trunk/lib/Passes/RoundTripYAMLPass.cpp Mon Jan 12 22:33:07 2015
> > > @@ -40,13 +40,15 @@ void RoundTripYAMLPass::perform(std::uni
> > > if (!mb)
> > > return;
> > >
> > > - std::error_code ec = _context.registry().parseFile(
> > > + std::error_code ec = _context.registry().loadFile(
> > > std::move(mb.get()), _yamlFile);
> > > if (ec) {
> > > // Note: we need a way for Passes to report errors.
> > > llvm_unreachable("yaml reader not registered or read error");
> > > }
> > > File *objFile = _yamlFile[0].get();
> > > + if (objFile->parse())
> > > + llvm_unreachable("native reader parse error");
> > > mergedFile.reset(new SimpleFileWrapper(_context, *objFile));
> > > llvm::sys::fs::remove(tmpYAMLFile.str());
> > > }
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/ELF/ELFReader.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/ELF/ELFReader.h?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/ELF/ELFReader.h (original)
> > > +++ lld/trunk/lib/ReaderWriter/ELF/ELFReader.h Mon Jan 12 22:33:07 2015
> > > @@ -33,8 +33,8 @@ public:
> > > }
> > >
> > > std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > - std::vector<std::unique_ptr<File>> &result) const override {
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > + std::vector<std::unique_ptr<File>> &result) const override {
> > > std::size_t maxAlignment =
> > > 1ULL << llvm::countTrailingZeros(uintptr_t(mb->getBufferStart()));
> > > auto f =
> > > @@ -72,8 +72,8 @@ public:
> > > }
> > >
> > > std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > - std::vector<std::unique_ptr<File>> &result) const override {
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > + std::vector<std::unique_ptr<File>> &result) const override {
> > > std::size_t maxAlignment =
> > > 1ULL << llvm::countTrailingZeros(uintptr_t(mb->getBufferStart()));
> > > auto f =
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/ELF/Mips/MipsELFReader.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/ELF/Mips/MipsELFReader.h?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/ELF/Mips/MipsELFReader.h (original)
> > > +++ lld/trunk/lib/ReaderWriter/ELF/Mips/MipsELFReader.h Mon Jan 12 22:33:07 2015
> > > @@ -49,12 +49,12 @@ public:
> > > _flagMerger(flagMerger) {}
> > >
> > > std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const Registry ®istry,
> > > - std::vector<std::unique_ptr<File>> &result) const override {
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const Registry ®istry,
> > > + std::vector<std::unique_ptr<File>> &result) const override {
> > > auto &hdr = *elfHeader(*mb);
> > > if (std::error_code ec = _flagMerger.merge(hdr.getFileClass(), hdr.e_flags))
> > > return ec;
> > > - return BaseReaderType::parseFile(std::move(mb), registry, result);
> > > + return BaseReaderType::loadFile(std::move(mb), registry, result);
> > > }
> > >
> > > private:
> > > @@ -72,12 +72,12 @@ public:
> > > _flagMerger(flagMerger) {}
> > >
> > > std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const Registry ®istry,
> > > - std::vector<std::unique_ptr<File>> &result) const override {
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const Registry ®istry,
> > > + std::vector<std::unique_ptr<File>> &result) const override {
> > > auto &hdr = *elfHeader(*mb);
> > > if (std::error_code ec = _flagMerger.merge(hdr.getFileClass(), hdr.e_flags))
> > > return ec;
> > > - return BaseReaderType::parseFile(std::move(mb), registry, result);
> > > + return BaseReaderType::loadFile(std::move(mb), registry, result);
> > > }
> > >
> > > private:
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/FileArchive.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/FileArchive.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/FileArchive.cpp (original)
> > > +++ lld/trunk/lib/ReaderWriter/FileArchive.cpp Mon Jan 12 22:33:07 2015
> > > @@ -67,7 +67,9 @@ public:
> > >
> > > /// \brief parse each member
> > > std::error_code
> > > - parseAllMembers(std::vector<std::unique_ptr<File>> &result) const override {
> > > + parseAllMembers(std::vector<std::unique_ptr<File>> &result) override {
> > > + if (std::error_code ec = parse())
> > > + return ec;
> > > for (auto mf = _archive->child_begin(), me = _archive->child_end();
> > > mf != me; ++mf) {
> > > std::unique_ptr<File> file;
> > > @@ -152,9 +154,12 @@ private:
> > > mb.getBuffer(), memberPath, false));
> > >
> > > std::vector<std::unique_ptr<File>> files;
> > > - _registry.parseFile(std::move(memberMB), files);
> > > + if (std::error_code ec = _registry.loadFile(std::move(memberMB), files))
> > > + return ec;
> > > assert(files.size() == 1);
> > > result = std::move(files[0]);
> > > + if (std::error_code ec = result->parse())
> > > + return ec;
> > >
> > > // The memory buffer is co-owned by the archive file and the children,
> > > // so that the bufffer is deallocated when all the members are destructed.
> > > @@ -232,8 +237,8 @@ public:
> > > }
> > >
> > > std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const Registry ®,
> > > - std::vector<std::unique_ptr<File>> &result) const override {
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const Registry ®,
> > > + std::vector<std::unique_ptr<File>> &result) const override {
> > > StringRef path = mb->getBufferIdentifier();
> > > std::unique_ptr<FileArchive> file(
> > > new FileArchive(std::move(mb), reg, path, _logLoading));
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/MachO/ExecutableAtoms.hpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/MachO/ExecutableAtoms.hpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/MachO/ExecutableAtoms.hpp (original)
> > > +++ lld/trunk/lib/ReaderWriter/MachO/ExecutableAtoms.hpp Mon Jan 12 22:33:07 2015
> > > @@ -91,7 +91,7 @@ public:
> > > }
> > >
> > > std::error_code
> > > - parseAllMembers(std::vector<std::unique_ptr<File>> &result) const override {
> > > + parseAllMembers(std::vector<std::unique_ptr<File>> &result) override {
> > > return std::error_code();
> > > }
> > >
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/MachO/MachOLinkingContext.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/MachO/MachOLinkingContext.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/MachO/MachOLinkingContext.cpp (original)
> > > +++ lld/trunk/lib/ReaderWriter/MachO/MachOLinkingContext.cpp Mon Jan 12 22:33:07 2015
> > > @@ -12,8 +12,8 @@
> > > #include "File.h"
> > > #include "MachONormalizedFile.h"
> > > #include "MachOPasses.h"
> > > +#include "lld/Core/ArchiveLibraryFile.h"
> > > #include "lld/Core/PassManager.h"
> > > -#include "lld/Driver/DarwinInputGraph.h"
> > > #include "lld/Passes/LayoutPass.h"
> > > #include "lld/Passes/RoundTripYAMLPass.h"
> > > #include "lld/ReaderWriter/Reader.h"
> > > @@ -605,20 +605,18 @@ Writer &MachOLinkingContext::writer() co
> > > }
> > >
> > > MachODylibFile* MachOLinkingContext::loadIndirectDylib(StringRef path) {
> > > - std::unique_ptr<MachOFileNode> node(new MachOFileNode(path, *this));
> > > - std::error_code ec = node->parse(*this, llvm::errs());
> > > - if (ec)
> > > + ErrorOr<std::unique_ptr<MemoryBuffer>> mbOrErr
> > > + = MemoryBuffer::getFileOrSTDIN(path);
> > > + if (mbOrErr.getError())
> > > return nullptr;
> > >
> > > - assert(node->files().size() == 1 && "expected one file in dylib");
> > > - // lld::File object is owned by MachOFileNode object. This method returns
> > > - // an unowned pointer to the lld::File object.
> > > - MachODylibFile* result = reinterpret_cast<MachODylibFile*>(
> > > - node->files().front().get());
> > > -
> > > + std::vector<std::unique_ptr<File>> files;
> > > + if (registry().loadFile(std::move(mbOrErr.get()), files))
> > > + return nullptr;
> > > + assert(files.size() == 1 && "expected one file in dylib");
> > > + MachODylibFile* result = reinterpret_cast<MachODylibFile*>(files[0].get());
> > > // Node object now owned by _indirectDylibs vector.
> > > - _indirectDylibs.push_back(std::move(node));
> > > -
> > > + _indirectDylibs.push_back(std::move(files[0]));
> > > return result;
> > > }
> > >
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFileBinaryReader.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFileBinaryReader.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFileBinaryReader.cpp (original)
> > > +++ lld/trunk/lib/ReaderWriter/MachO/MachONormalizedFileBinaryReader.cpp Mon Jan 12 22:33:07 2015
> > > @@ -527,8 +527,8 @@ public:
> > > }
> > >
> > > std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const Registry ®istry,
> > > - std::vector<std::unique_ptr<File>> &result) const override {
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const Registry ®istry,
> > > + std::vector<std::unique_ptr<File>> &result) const override {
> > > auto *file = new MachOFile(std::move(mb), &_ctx);
> > > result.push_back(std::unique_ptr<MachOFile>(file));
> > > return std::error_code();
> > > @@ -554,8 +554,8 @@ public:
> > > }
> > >
> > > std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const Registry ®istry,
> > > - std::vector<std::unique_ptr<File>> &result) const override {
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const Registry ®istry,
> > > + std::vector<std::unique_ptr<File>> &result) const override {
> > > auto *file = new MachODylibFile(std::move(mb), &_ctx);
> > > result.push_back(std::unique_ptr<MachODylibFile>(file));
> > > return std::error_code();
> > > @@ -580,4 +580,3 @@ void Registry::addSupportMachOObjects(Ma
> > >
> > >
> > > } // namespace lld
> > > -
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/Native/ReaderNative.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/Native/ReaderNative.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/Native/ReaderNative.cpp (original)
> > > +++ lld/trunk/lib/ReaderWriter/Native/ReaderNative.cpp Mon Jan 12 22:33:07 2015
> > > @@ -999,8 +999,8 @@ public:
> > > }
> > >
> > > virtual std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > - std::vector<std::unique_ptr<File>> &result) const override {
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > + std::vector<std::unique_ptr<File>> &result) const override {
> > > auto *file = new lld::native::File(std::move(mb));
> > > result.push_back(std::unique_ptr<File>(file));
> > > return std::error_code();
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/PECOFF/LinkerGeneratedSymbolFile.h
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/PECOFF/LinkerGeneratedSymbolFile.h?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/PECOFF/LinkerGeneratedSymbolFile.h (original)
> > > +++ lld/trunk/lib/ReaderWriter/PECOFF/LinkerGeneratedSymbolFile.h Mon Jan 12 22:33:07 2015
> > > @@ -88,7 +88,7 @@ public:
> > > }
> > >
> > > std::error_code
> > > - parseAllMembers(std::vector<std::unique_ptr<File>> &result) const override {
> > > + parseAllMembers(std::vector<std::unique_ptr<File>> &result) override {
> > > return std::error_code();
> > > }
> > >
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/PECOFF/ReaderCOFF.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/PECOFF/ReaderCOFF.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/PECOFF/ReaderCOFF.cpp (original)
> > > +++ lld/trunk/lib/ReaderWriter/PECOFF/ReaderCOFF.cpp Mon Jan 12 22:33:07 2015
> > > @@ -1077,8 +1077,8 @@ public:
> > > }
> > >
> > > std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const Registry &,
> > > - std::vector<std::unique_ptr<File>> &result) const override {
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const Registry &,
> > > + std::vector<std::unique_ptr<File>> &result) const override {
> > > // Parse the memory buffer as PECOFF file.
> > > auto *file = new FileCOFF(std::move(mb), _ctx);
> > > result.push_back(std::unique_ptr<File>(file));
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/PECOFF/ReaderImportHeader.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/PECOFF/ReaderImportHeader.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/PECOFF/ReaderImportHeader.cpp (original)
> > > +++ lld/trunk/lib/ReaderWriter/PECOFF/ReaderImportHeader.cpp Mon Jan 12 22:33:07 2015
> > > @@ -364,8 +364,8 @@ public:
> > > }
> > >
> > > std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > - std::vector<std::unique_ptr<File> > &result) const override {
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > + std::vector<std::unique_ptr<File> > &result) const override {
> > > auto *file = new FileImportLibrary(std::move(mb), _machine);
> > > result.push_back(std::unique_ptr<File>(file));
> > > return std::error_code();
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/Reader.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/Reader.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/Reader.cpp (original)
> > > +++ lld/trunk/lib/ReaderWriter/Reader.cpp Mon Jan 12 22:33:07 2015
> > > @@ -30,8 +30,8 @@ void Registry::add(std::unique_ptr<YamlI
> > > }
> > >
> > > std::error_code
> > > -Registry::parseFile(std::unique_ptr<MemoryBuffer> mb,
> > > - std::vector<std::unique_ptr<File>> &result) const {
> > > +Registry::loadFile(std::unique_ptr<MemoryBuffer> mb,
> > > + std::vector<std::unique_ptr<File>> &result) const {
> > > // Get file type.
> > > StringRef content(mb->getBufferStart(), mb->getBufferSize());
> > > llvm::sys::fs::file_magic fileType = llvm::sys::fs::identify_magic(content);
> > > @@ -42,11 +42,8 @@ Registry::parseFile(std::unique_ptr<Memo
> > > for (const std::unique_ptr<Reader> &reader : _readers) {
> > > if (!reader->canParse(fileType, extension, *mb))
> > > continue;
> > > - if (std::error_code ec = reader->parseFile(std::move(mb), *this, result))
> > > + if (std::error_code ec = reader->loadFile(std::move(mb), *this, result))
> > > return ec;
> > > - for (std::unique_ptr<File> &file : result)
> > > - if (std::error_code ec = file->parse())
> > > - return ec;
> > > return std::error_code();
> > > }
> > >
> > >
> > > Modified: lld/trunk/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp (original)
> > > +++ lld/trunk/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp Mon Jan 12 22:33:07 2015
> > > @@ -648,7 +648,7 @@ template <> struct MappingTraits<const l
> > > }
> > >
> > > virtual std::error_code
> > > - parseAllMembers(std::vector<std::unique_ptr<File>> &result) const override {
> > > + parseAllMembers(std::vector<std::unique_ptr<File>> &result) override {
> > > return std::error_code();
> > > }
> > >
> > > @@ -1325,8 +1325,8 @@ public:
> > > }
> > >
> > > std::error_code
> > > - parseFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > - std::vector<std::unique_ptr<File>> &result) const override {
> > > + loadFile(std::unique_ptr<MemoryBuffer> mb, const class Registry &,
> > > + std::vector<std::unique_ptr<File>> &result) const override {
> > > // Create YAML Input Reader.
> > > YamlContext yamlContext;
> > > yamlContext._registry = &_registry;
> > > @@ -1343,7 +1343,7 @@ public:
> > >
> > > std::shared_ptr<MemoryBuffer> smb(mb.release());
> > > for (const File *file : createdFiles) {
> > > - // Note: parseFile() should return vector of *const* File
> > > + // Note: loadFile() should return vector of *const* File
> > > File *f = const_cast<File *>(file);
> > > f->setLastError(std::error_code());
> > > f->setSharedMemoryBuffer(smb);
> > >
> > > Removed: lld/trunk/test/Driver/libsearch-inputGraph.test
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/test/Driver/libsearch-inputGraph.test?rev=225763&view=auto
> > > ==============================================================================
> > > --- lld/trunk/test/Driver/libsearch-inputGraph.test (original)
> > > +++ lld/trunk/test/Driver/libsearch-inputGraph.test (removed)
> > > @@ -1,39 +0,0 @@
> > > -RUN: lld -flavor gnu -target x86_64-linux -L%p/../elf/Inputs -lfnarchive \
> > > -RUN: --output-filetype=yaml --noinhibit-exec 2> %t.err
> > > -RUN: FileCheck %s < %t.err
> > > -
> > > -RUN: lld -flavor gnu -target x86_64-linux -L%p/../elf/Inputs --whole-archive \
> > > -RUN: -lfnarchive --output-filetype=yaml --noinhibit-exec 2> %t1.err
> > > -RUN: FileCheck %s -check-prefix="WHOLEARCHIVE" < %t1.err
> > > -
> > > -RUN: lld -flavor gnu -target x86_64-linux -L%p/../elf/Inputs --whole-archive \
> > > -RUN: --as-needed -lfnarchive --output-filetype=yaml --noinhibit-exec 2> %t2.err
> > > -RUN: FileCheck %s -check-prefix="ASNEEDED" < %t2.err
> > > -
> > > -RUN: lld -flavor gnu -target x86_64-linux --sysroot=%p/../elf -L=/Inputs \
> > > -RUN: -lfnarchive --output-filetype=yaml --noinhibit-exec 2> %t3.err
> > > -RUN: FileCheck -check-prefix="SYSROOT" %s < %t3.err
> > > -
> > > -CHECK: Name : {{[^ ]+}}elf/Inputs{{[\\/]}}libfnarchive.a
> > > -CHECK: Type : ELF File
> > > -CHECK: Attributes :
> > > -CHECK: - wholeArchive : false
> > > -CHECK: - asNeeded : false
> > > -
> > > -WHOLEARCHIVE: Name : {{[^ ]+}}elf/Inputs{{[\\/]}}libfnarchive.a
> > > -WHOLEARCHIVE: Type : ELF File
> > > -WHOLEARCHIVE: Attributes :
> > > -WHOLEARCHIVE: - wholeArchive : true
> > > -WHOLEARCHIVE: - asNeeded : false
> > > -
> > > -ASNEEDED: Name : {{[^ ]+}}elf/Inputs{{[\\/]}}libfnarchive.a
> > > -ASNEEDED: Type : ELF File
> > > -ASNEEDED: Attributes :
> > > -ASNEEDED: - wholeArchive : true
> > > -ASNEEDED: - asNeeded : true
> > > -
> > > -SYSROOT: Name : {{[^ ]+}}elf/Inputs{{[\\/]}}libfnarchive.a
> > > -SYSROOT: Type : ELF File
> > > -SYSROOT: Attributes :
> > > -SYSROOT: - wholeArchive : false
> > > -SYSROOT: - asNeeded : false
> > >
> > > Modified: lld/trunk/test/pecoff/hello64.test
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/test/pecoff/hello64.test?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/test/pecoff/hello64.test (original)
> > > +++ lld/trunk/test/pecoff/hello64.test Mon Jan 12 22:33:07 2015
> > > @@ -12,9 +12,9 @@ CHECK: 6019: 41 b9 00 00 00 00 mov
> > > CHECK: 601f: e8 0a 00 00 00 callq 10
> > > CHECK: 6024: b9 00 00 00 00 movl $0, %ecx
> > > CHECK: 6029: e8 08 00 00 00 callq 8
> > > -CHECK: 602e: ff 25 d4 cf ff ff jmpq *-12332(%rip)
> > > +CHECK: 602e: ff 25 d2 cf ff ff jmpq *-12334(%rip)
> > > CHECK: 6034: cc int3
> > > CHECK: 6035: cc int3
> > > -CHECK: 6036: ff 25 c4 cf ff ff jmpq *-12348(%rip)
> > > +CHECK: 6036: ff 25 c2 cf ff ff jmpq *-12350(%rip)
> > > CHECK: 603c: cc int3
> > > CHECK: 603d: cc int3
> > >
> > > Modified: lld/trunk/unittests/DriverTests/WinLinkDriverTest.cpp
> > > URL: http://llvm.org/viewvc/llvm-project/lld/trunk/unittests/DriverTests/WinLinkDriverTest.cpp?rev=225764&r1=225763&r2=225764&view=diff
> > > ==============================================================================
> > > --- lld/trunk/unittests/DriverTests/WinLinkDriverTest.cpp (original)
> > > +++ lld/trunk/unittests/DriverTests/WinLinkDriverTest.cpp Mon Jan 12 22:33:07 2015
> > > @@ -698,10 +698,9 @@ TEST_F(WinLinkParserTest, Ignore) {
> > > // compatibility with link.exe.
> > > EXPECT_TRUE(parse("link.exe", "/nologo", "/errorreport:prompt",
> > > "/incremental", "/incremental:no", "/delay:unload",
> > > - "/disallowlib:foo", "/pdbaltpath:bar", "/verbose",
> > > - "/verbose:icf", "/wx", "/wx:no", "/tlbid:1",
> > > - "/tlbout:foo", "/idlout:foo", "/ignore:4000",
> > > - "/ignoreidl", "/implib:foo", "/safeseh",
> > > + "/disallowlib:foo", "/pdbaltpath:bar",
> > > + "/wx", "/wx:no", "/tlbid:1", "/tlbout:foo", "/idlout:foo",
> > > + "/ignore:4000", "/ignoreidl", "/implib:foo", "/safeseh",
> > > "/safeseh:no", "/functionpadmin", "/maxilksize:1024",
> > > "a.obj", nullptr));
> > > EXPECT_EQ("", errorMessage());
> > >
> > >
> > > _______________________________________________
> > > llvm-commits mailing list
> > > llvm-commits at cs.uiuc.edu
> > > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
More information about the llvm-commits
mailing list