Use error_code() instead of error_code::succes()
authorRafael Espindola <rafael.espindola@gmail.com>
Sat, 31 May 2014 01:22:21 +0000 (01:22 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Sat, 31 May 2014 01:22:21 +0000 (01:22 +0000)
There is no std::error_code::success, so this removes much of the noise
in transitioning to std::error_code.

llvm-svn: 209948

36 files changed:
lld/include/lld/Core/InputGraph.h
lld/include/lld/Driver/DarwinInputGraph.h
lld/lib/Core/InputGraph.cpp
lld/lib/Core/PassManager.cpp
lld/lib/Driver/GnuLdDriver.cpp
lld/lib/Driver/GnuLdInputGraph.cpp
lld/lib/Driver/WinLinkInputGraph.cpp
lld/lib/ReaderWriter/ELF/ELFFile.h
lld/lib/ReaderWriter/ELF/ELFReader.h
lld/lib/ReaderWriter/ELF/Hexagon/HexagonDynamicLibraryWriter.h
lld/lib/ReaderWriter/ELF/Hexagon/HexagonELFReader.h
lld/lib/ReaderWriter/ELF/Hexagon/HexagonExecutableWriter.h
lld/lib/ReaderWriter/ELF/Hexagon/HexagonRelocationHandler.cpp
lld/lib/ReaderWriter/ELF/Hexagon/HexagonTargetHandler.cpp
lld/lib/ReaderWriter/ELF/Mips/MipsDynamicLibraryWriter.h
lld/lib/ReaderWriter/ELF/Mips/MipsELFFile.h
lld/lib/ReaderWriter/ELF/Mips/MipsELFReader.h
lld/lib/ReaderWriter/ELF/Mips/MipsExecutableWriter.h
lld/lib/ReaderWriter/ELF/Mips/MipsRelocationHandler.cpp
lld/lib/ReaderWriter/ELF/OutputELFWriter.h
lld/lib/ReaderWriter/ELF/PPC/PPCELFReader.h
lld/lib/ReaderWriter/ELF/PPC/PPCTargetHandler.cpp
lld/lib/ReaderWriter/ELF/X86/X86TargetHandler.cpp
lld/lib/ReaderWriter/ELF/X86_64/X86_64RelocationHandler.cpp
lld/lib/ReaderWriter/ELF/X86_64/X86_64RelocationPass.cpp
lld/lib/ReaderWriter/FileArchive.cpp
lld/lib/ReaderWriter/MachO/MachONormalizedFileBinaryReader.cpp
lld/lib/ReaderWriter/MachO/MachONormalizedFileBinaryWriter.cpp
lld/lib/ReaderWriter/MachO/MachONormalizedFileToAtoms.cpp
lld/lib/ReaderWriter/MachO/MachONormalizedFileYAML.cpp
lld/lib/ReaderWriter/Native/ReaderNative.cpp
lld/lib/ReaderWriter/Native/WriterNative.cpp
lld/lib/ReaderWriter/PECOFF/LinkerGeneratedSymbolFile.h
lld/lib/ReaderWriter/PECOFF/ReaderCOFF.cpp
lld/lib/ReaderWriter/PECOFF/ReaderImportHeader.cpp
lld/lib/ReaderWriter/YAML/ReaderWriterYAML.cpp

index 058bf92..20a91d0 100644 (file)
@@ -188,7 +188,7 @@ public:
     for (std::unique_ptr<InputElement> &ei : _elements)
       if (error_code ec = ei->parse(ctx, diag))
         return ec;
-    return error_code::success();
+    return error_code();
   }
 
   /// If Resolver made a progress using the current file, it's ok to revisit
@@ -290,7 +290,7 @@ public:
 
   /// \brief parse the input element
   error_code parse(const LinkingContext &, raw_ostream &) override {
-    return error_code::success();
+    return error_code();
   }
 
   /// \brief Return the next File thats part of this node to the
index e918434..0e74aec 100644 (file)
@@ -58,7 +58,7 @@ public:
       } else {
         // if --whole-archive is around non-archive, just use it as normal.
         _files.push_back(std::move(f));
-        return error_code::success();
+        return error_code();
       }
     }
     return ctx.registry().parseFile(_buffer, _files);
index abd3385..eee6faf 100644 (file)
@@ -90,7 +90,7 @@ error_code FileNode::getBuffer(StringRef filePath) {
   if (error_code ec = MemoryBuffer::getFileOrSTDIN(filePath, mb))
     return ec;
   _buffer = std::move(mb);
-  return error_code::success();
+  return error_code();
 }
 
 /// \brief Return the next file that need to be processed by the resolver.
index c461461..eca7ce1 100644 (file)
@@ -18,6 +18,6 @@ namespace lld {
 error_code PassManager::runOnFile(std::unique_ptr<MutableFile> &mf) {
   for (std::unique_ptr<Pass> &pass : _passes)
     pass->perform(mf);
-  return error_code::success();
+  return error_code();
 }
 } // end namespace lld
index 9f18d57..ef5bfa8 100644 (file)
@@ -122,7 +122,7 @@ static error_code getFileMagic(ELFLinkingContext &ctx, StringRef path,
   case llvm::sys::fs::file_magic::elf_relocatable:
   case llvm::sys::fs::file_magic::elf_shared_object:
   case llvm::sys::fs::file_magic::unknown:
-    return error_code::success();
+    return error_code();
   default:
     break;
   }
index fd80606..a3c6543 100644 (file)
@@ -41,7 +41,7 @@ error_code ELFFileNode::parse(const LinkingContext &ctx,
     }
     // if --whole-archive is around non-archive, just use it as normal.
     _files.push_back(std::move(f));
-    return error_code::success();
+    return error_code();
   }
   return ctx.registry().parseFile(_buffer, _files);
 }
@@ -66,7 +66,7 @@ error_code GNULdScript::parse(const LinkingContext &ctx,
   if (!_linkerScript)
     return LinkerScriptReaderError::parse_error;
 
-  return error_code::success();
+  return error_code();
 }
 
 static bool isPathUnderSysroot(StringRef sysroot, StringRef path) {
@@ -106,5 +106,5 @@ error_code ELFGNULdScript::parse(const LinkingContext &ctx,
     }
     _expandElements.push_back(std::move(groupStart));
   }
-  return error_code::success();
+  return error_code();
 }
index 918ea58..fd6f5b5 100644 (file)
@@ -19,7 +19,7 @@ bool isCOFFLibraryFileExtension(StringRef path) {
 error_code PECOFFFileNode::parse(const LinkingContext &ctx,
                                  raw_ostream &diagnostics) {
   if (_parsed)
-    return error_code::success();
+    return error_code();
   _parsed = true;
   ErrorOr<StringRef> filePath = getPath(ctx);
   if (error_code ec = filePath.getError()) {
index 4e63009..65cc894 100644 (file)
@@ -509,7 +509,7 @@ template <class ELFT> error_code ELFFile<ELFT>::createAtomizableSections() {
     }
   }
   _references.reserve(totalRelocs);
-  return error_code::success();
+  return error_code();
 }
 
 template <class ELFT> error_code ELFFile<ELFT>::createMergeableAtoms() {
@@ -550,7 +550,7 @@ template <class ELFT> error_code ELFFile<ELFT>::createMergeableAtoms() {
     _definedAtoms._atoms.push_back(*mergeAtom);
     _mergeAtoms.push_back(*mergeAtom);
   }
-  return error_code::success();
+  return error_code();
 }
 
 template <class ELFT>
@@ -594,7 +594,7 @@ error_code ELFFile<ELFT>::createSymbolsFromAtomizableSections() {
     }
   }
 
-  return error_code::success();
+  return error_code();
 }
 
 template <class ELFT> error_code ELFFile<ELFT>::createAtoms() {
@@ -736,7 +736,7 @@ template <class ELFT> error_code ELFFile<ELFT>::createAtoms() {
   }
 
   updateReferences();
-  return error_code::success();
+  return error_code();
 }
 
 template <class ELFT>
index ecf9741..932649e 100644 (file)
@@ -59,7 +59,7 @@ public:
     if (error_code ec = f.getError())
       return ec;
     result.push_back(std::move(*f));
-    return error_code::success();
+    return error_code();
   }
 
 protected:
@@ -86,7 +86,7 @@ public:
     if (error_code ec = f.getError())
       return ec;
     result.push_back(std::move(*f));
-    return error_code::success();
+    return error_code();
   }
 
 protected:
index 0cbb96e..34e517a 100644 (file)
@@ -34,7 +34,7 @@ protected:
   virtual error_code setELFHeader() {
     DynamicLibraryWriter<ELFT>::setELFHeader();
     HexagonELFWriter<ELFT>::setELFHeader(*this->_elfHeader);
-    return error_code::success();
+    return error_code();
   }
 
 private:
index 9dda08a..d99a74f 100644 (file)
@@ -54,7 +54,7 @@ public:
     if (error_code ec = f.getError())
       return ec;
     result.push_back(std::move(*f));
-    return error_code::success();
+    return error_code();
   }
 };
 
@@ -73,7 +73,7 @@ public:
     if (error_code ec = f.getError())
       return ec;
     result.push_back(std::move(*f));
-    return error_code::success();
+    return error_code();
   }
 };
 
index 291b2d6..b00091a 100644 (file)
@@ -35,7 +35,7 @@ protected:
   virtual error_code setELFHeader() {
     ExecutableWriter<ELFT>::setELFHeader();
     HexagonELFWriter<ELFT>::setELFHeader(*this->_elfHeader);
-    return error_code::success();
+    return error_code();
   }
 
 private:
index 22f10f7..89402ea 100644 (file)
@@ -220,7 +220,7 @@ error_code HexagonTargetRelocationHandler::applyRelocation(
   uint64_t relocVAddress = atom._virtualAddr + ref.offsetInAtom();
 
   if (ref.kindNamespace() != Reference::KindNamespace::ELF)
-    return error_code::success();
+    return error_code();
   assert(ref.kindArch() == Reference::KindArch::Hexagon);
   switch (ref.kindValue()) {
   case R_HEX_B22_PCREL:
@@ -353,7 +353,7 @@ error_code HexagonTargetRelocationHandler::applyRelocation(
   }
   }
 
-  return error_code::success();
+  return error_code();
 }
 
 
index 7f7a855..58f3f73 100644 (file)
@@ -304,7 +304,7 @@ public:
   error_code handleGOTREL(const Reference &ref) {
     // Turn this so that the target is set to the GOT entry
     const_cast<Reference &>(ref).setTarget(getGOTEntry(ref.target()));
-    return error_code::success();
+    return error_code();
   }
 
   error_code handlePLT32(const Reference &ref) {
@@ -313,7 +313,7 @@ public:
     assert(ref.kindArch() == Reference::KindArch::Hexagon);
     const_cast<Reference &>(ref).setKindValue(R_HEX_B22_PCREL);
     const_cast<Reference &>(ref).setTarget(getPLTEntry(ref.target()));
-    return error_code::success();
+    return error_code();
   }
 };
 
index ed889db..f1e47e7 100644 (file)
@@ -34,7 +34,7 @@ protected:
   error_code setELFHeader() override {
     DynamicLibraryWriter<ELFT>::setELFHeader();
     _writeHelper.setELFHeader(*this->_elfHeader);
-    return error_code::success();
+    return error_code();
   }
 
   LLD_UNIQUE_BUMP_PTR(DynamicTable<ELFT>) createDynamicTable();
index f908d12..afdfc11 100644 (file)
@@ -158,7 +158,7 @@ private:
       } else if (!_tpOff.hasValue() && section.sh_flags & llvm::ELF::SHF_TLS)
         _tpOff = section.sh_addr + TP_OFFSET;
     }
-    return error_code::success();
+    return error_code();
   }
 
   void createRelocationReferences(const Elf_Sym &symbol,
index a03d605..f58332f 100644 (file)
@@ -40,7 +40,7 @@ public:
     if (error_code ec = f.getError())
       return ec;
     result.push_back(std::move(*f));
-    return error_code::success();
+    return error_code();
   }
 };
 
index 27997f6..65034d2 100644 (file)
@@ -36,7 +36,7 @@ protected:
   error_code setELFHeader() override {
     ExecutableWriter<ELFT>::setELFHeader();
     _writeHelper.setELFHeader(*this->_elfHeader);
-    return error_code::success();
+    return error_code();
   }
 
   LLD_UNIQUE_BUMP_PTR(DynamicTable<ELFT>) createDynamicTable();
index ce92041..a1e0ea3 100644 (file)
@@ -123,7 +123,7 @@ error_code MipsTargetRelocationHandler::applyRelocation(
     ELFWriter &writer, llvm::FileOutputBuffer &buf, const lld::AtomLayout &atom,
     const Reference &ref) const {
   if (ref.kindNamespace() != lld::Reference::KindNamespace::ELF)
-    return error_code::success();
+    return error_code();
   assert(ref.kindArch() == Reference::KindArch::Mips);
 
   AtomLayout *gpAtom = _mipsTargetLayout.getGP();
@@ -204,5 +204,5 @@ error_code MipsTargetRelocationHandler::applyRelocation(
   }
   }
 
-  return error_code::success();
+  return error_code();
 }
index 75f5882..394695e 100644 (file)
@@ -394,7 +394,7 @@ error_code OutputELFWriter<ELFT>::buildOutput(const File &file) {
   if (_context.isDynamic())
     _dynamicTable->updateDynamicTable();
 
-  return error_code::success();
+  return error_code();
 }
 
 template <class ELFT> error_code OutputELFWriter<ELFT>::setELFHeader() {
@@ -419,7 +419,7 @@ template <class ELFT> error_code OutputELFWriter<ELFT>::setELFHeader() {
   _layout.findAtomAddrByName(_context.entrySymbolName(), virtualAddr);
   _elfHeader->e_entry(virtualAddr);
 
-  return error_code::success();
+  return error_code();
 }
 
 template <class ELFT> uint64_t OutputELFWriter<ELFT>::outputFileSize() const {
index 41d95f3..159b6da 100644 (file)
@@ -52,7 +52,7 @@ public:
     if (!f)
       return f;
     result.push_back(std::move(*f));
-    return error_code::success();
+    return error_code();
   }
 };
 
@@ -71,7 +71,7 @@ public:
     if (!f)
       return f;
     result.push_back(std::move(*f));
-    return error_code::success();
+    return error_code();
   }
 };
 
index 8f05bb9..f76b169 100644 (file)
@@ -44,7 +44,7 @@ error_code PPCTargetRelocationHandler::applyRelocation(
   uint64_t relocVAddress = atom._virtualAddr + ref.offsetInAtom();
 
   if (ref.kindNamespace() != Reference::KindNamespace::ELF)
-    return error_code::success();
+    return error_code();
   assert(ref.kindArch() == Reference::KindArch::PowerPC);
   switch (ref.kindValue()) {
   case R_PPC_REL24:
@@ -60,7 +60,7 @@ error_code PPCTargetRelocationHandler::applyRelocation(
   }
   }
 
-  return error_code::success();
+  return error_code();
 }
 
 PPCTargetHandler::PPCTargetHandler(PPCLinkingContext &context)
index 367b4d8..25ad7a8 100644 (file)
@@ -105,7 +105,7 @@ error_code X86TargetRelocationHandler::applyRelocation(
   uint64_t relocVAddress = atom._virtualAddr + ref.offsetInAtom();
 
   if (ref.kindNamespace() != Reference::KindNamespace::ELF)
-    return error_code::success();
+    return error_code();
   assert(ref.kindArch() == Reference::KindArch::x86);
   switch (ref.kindValue()) {
   case R_386_32:
@@ -123,7 +123,7 @@ error_code X86TargetRelocationHandler::applyRelocation(
   }
   }
 
-  return error_code::success();
+  return error_code();
 }
 
 X86TargetHandler::X86TargetHandler(X86LinkingContext &context)
index 750b9d1..1b800a5 100644 (file)
@@ -69,7 +69,7 @@ error_code X86_64TargetRelocationHandler::applyRelocation(
   uint64_t relocVAddress = atom._virtualAddr + ref.offsetInAtom();
 
   if (ref.kindNamespace() != Reference::KindNamespace::ELF)
-    return error_code::success();
+    return error_code();
   assert(ref.kindArch() == Reference::KindArch::x86_64);
   switch (ref.kindValue()) {
   case R_X86_64_NONE:
@@ -140,5 +140,5 @@ error_code X86_64TargetRelocationHandler::applyRelocation(
   }
   }
 
-  return error_code::success();
+  return error_code();
 }
index e108751..f03b5be 100644 (file)
@@ -153,7 +153,7 @@ protected:
     auto target = dyn_cast_or_null<const DefinedAtom>(ref.target());
     if (target && target->contentType() == DefinedAtom::typeResolver)
       const_cast<Reference &>(ref).setTarget(getIFUNCPLTEntry(target));
-    return error_code::success();
+    return error_code();
   }
 
   /// \brief Create a GOT entry for the TP offset of a TLS atom.
@@ -306,7 +306,7 @@ public:
     // __tls_get_addr is handled elsewhere.
     if (ref.target() && ref.target()->name() == "__tls_get_addr") {
       const_cast<Reference &>(ref).setKindValue(R_X86_64_NONE);
-      return error_code::success();
+      return error_code();
     }
     // Static code doesn't need PLTs.
     const_cast<Reference &>(ref).setKindValue(R_X86_64_PC32);
@@ -315,7 +315,7 @@ public:
             dyn_cast_or_null<const DefinedAtom>(ref.target()))
       if (da->contentType() == DefinedAtom::typeResolver)
         return handleIFUNC(ref);
-    return error_code::success();
+    return error_code();
   }
 
   error_code handleGOT(const Reference &ref) {
@@ -323,7 +323,7 @@ public:
       const_cast<Reference &>(ref).setTarget(getNullGOT());
     else if (const DefinedAtom *da = dyn_cast<const DefinedAtom>(ref.target()))
       const_cast<Reference &>(ref).setTarget(getGOT(da));
-    return error_code::success();
+    return error_code();
   }
 };
 
@@ -395,7 +395,7 @@ public:
 
   error_code handlePlain(const Reference &ref) {
     if (!ref.target())
-      return error_code::success();
+      return error_code();
     if (auto sla = dyn_cast<SharedLibraryAtom>(ref.target())) {
       if (sla->type() == SharedLibraryAtom::Type::Data)
         const_cast<Reference &>(ref).setTarget(getObjectEntry(sla));
@@ -403,7 +403,7 @@ public:
         const_cast<Reference &>(ref).setTarget(getPLTEntry(sla));
     } else
       return handleIFUNC(ref);
-    return error_code::success();
+    return error_code();
   }
 
   error_code handlePLT32(const Reference &ref) {
@@ -416,7 +416,7 @@ public:
         return handleIFUNC(ref);
     if (isa<const SharedLibraryAtom>(ref.target()))
       const_cast<Reference &>(ref).setTarget(getPLTEntry(ref.target()));
-    return error_code::success();
+    return error_code();
   }
 
   const GOTAtom *getSharedGOT(const SharedLibraryAtom *sla) {
@@ -442,7 +442,7 @@ public:
       const_cast<Reference &>(ref).setTarget(getGOT(da));
     else if (const auto sla = dyn_cast<const SharedLibraryAtom>(ref.target()))
       const_cast<Reference &>(ref).setTarget(getSharedGOT(sla));
-    return error_code::success();
+    return error_code();
   }
 };
 } // end anon namespace
index 086e909..140d43e 100644 (file)
@@ -78,7 +78,7 @@ public:
       if (error_code ec = instantiateMember(mf, result))
         return ec;
     }
-    return error_code::success();
+    return error_code();
   }
 
   const atom_collection<DefinedAtom> &defined() const override {
@@ -117,7 +117,7 @@ protected:
     _registry.parseFile(mb, result);
     const char *memberStart = member->getBuffer().data();
     _membersInstantiated.insert(memberStart);
-    return error_code::success();
+    return error_code();
   }
 
   error_code isDataSymbol(std::unique_ptr<MemoryBuffer> mb, StringRef symbol) const {
@@ -152,7 +152,7 @@ protected:
         return ec;
 
       if (symtype == SymbolRef::ST_Data) {
-        return error_code::success();
+        return error_code();
       }
     }
     return object_error::parse_failed;
@@ -199,7 +199,7 @@ public:
                        << "'" << name << "'\n");
       _symbolMemberMap[name] = member;
     }
-    return error_code::success();
+    return error_code();
   }
 
 }; // class FileArchive
@@ -233,7 +233,7 @@ public:
     mb.release();
 
     result.push_back(std::move(file));
-    return error_code::success();
+    return error_code();
   }
 
 private:
index bb1b5dd..ce10867 100644 (file)
@@ -68,12 +68,12 @@ forEachLoadCommand(StringRef lcRange, unsigned lcCount, bool swap, bool is64,
       return llvm::make_error_code(llvm::errc::executable_format_error);
 
     if (func(slc->cmd, slc->cmdsize, p))
-      return error_code::success();
+      return error_code();
 
     p += slc->cmdsize;
   }
 
-  return error_code::success();
+  return error_code();
 }
 
 
@@ -88,7 +88,7 @@ appendRelocations(Relocations &relocs, StringRef buffer, bool swap,
   for(uint32_t i=0; i < nreloc; ++i) {
     relocs.push_back(unpackRelocation(relocsArray[i], swap, bigEndian));
   }
-  return error_code::success();
+  return error_code();
 }
 
 static error_code
@@ -105,7 +105,7 @@ appendIndirectSymbols(IndirectSymbols &isyms, StringRef buffer, bool swap,
   for(uint32_t i=0; i < count; ++i) {
     isyms.push_back(read32(swap, indirectSymbolArray[startIndex+i]));
   }
-  return error_code::success();
+  return error_code();
 }
 
 
@@ -426,7 +426,7 @@ public:
 
     result.push_back(std::move(*file));
 
-    return error_code::success();
+    return error_code();
   }
 private:
   MachOLinkingContext::Arch _arch;
index 1012319..4a39a4b 100644 (file)
@@ -524,7 +524,7 @@ error_code MachOFileLayout::writeSingleSegmentLoadCommand(uint8_t *&lc) {
     ++sout;
   }
   lc = next;
-  return error_code::success();
+  return error_code();
 }
 
 
@@ -587,7 +587,7 @@ error_code MachOFileLayout::writeSegmentLoadCommands(uint8_t *&lc) {
   if (_swap)
     swapStruct(*cmd);
   lc = next;
-  return error_code::success();
+  return error_code();
 }
 
 
@@ -954,7 +954,7 @@ error_code MachOFileLayout::writeBinary(StringRef path) {
   writeLinkEditContent();
   fob->commit();
 
-  return error_code::success();
+  return error_code();
 }
 
 
index 4f2edbc..a81c74b 100644 (file)
@@ -120,7 +120,7 @@ processSymbol(const NormalizedFile &normalizedFile, MachOFile &file,
   case symbolsOk:
     break;
   case symbolsIgnored:
-    return error_code::success();
+    return error_code();
     break;
   case symbolsIllegal:
     return make_dynamic_error_code(Twine("Symbol '") + sym.name
@@ -155,7 +155,7 @@ processSymbol(const NormalizedFile &normalizedFile, MachOFile &file,
                         copyRefs);
     }
   }
-  return error_code::success();
+  return error_code();
 }
 
 
@@ -198,7 +198,7 @@ static error_code processUTF16Section(MachOFile &file, const Section &section,
                                    "last string in the section is not zero "
                                    "terminated.");
   }
-  return error_code::success();
+  return error_code();
 }
 
 // A __DATA/__cfstring section contain NS/CFString objects. Atom boundaries
@@ -220,7 +220,7 @@ static error_code processCFStringSection(MachOFile &file,const Section &section,
                         DefinedAtom::mergeByContent, byteContent, copyRefs);
     offset += cfsObjSize;
   }
-  return error_code::success();
+  return error_code();
 }
 
 
@@ -254,7 +254,7 @@ static error_code processCFISection(MachOFile &file, const Section &section,
                         bytes, copyRefs);
     offset += len;
   }
-  return error_code::success();
+  return error_code();
 }
 
 static error_code 
@@ -275,7 +275,7 @@ processCompactUnwindSection(MachOFile &file, const Section &section,
                         DefinedAtom::mergeNo, byteContent, copyRefs);
     offset += cuObjSize;
   }
-  return error_code::success();
+  return error_code();
 }
 
 static error_code processSection(MachOFile &file, const Section &section,
@@ -437,7 +437,7 @@ static error_code processSection(MachOFile &file, const Section &section,
     llvm_unreachable("mach-o section type not supported yet");
     break;
   }
-  return error_code::success();
+  return error_code();
 }
 
 static ErrorOr<std::unique_ptr<lld::File>>
index 4ce61fc..d792975 100644 (file)
@@ -706,7 +706,7 @@ writeYaml(const NormalizedFile &file, raw_ostream &out) {
   // Stream out yaml.
   yout << *f;
 
-  return error_code::success();
+  return error_code();
 }
 
 } // namespace normalized
index 9b8a749..e6871ae 100644 (file)
@@ -1009,7 +1009,7 @@ public:
   parseFile(std::unique_ptr<MemoryBuffer> &mb, const class Registry &,
             std::vector<std::unique_ptr<File>> &result) const override {
     return lld::native::File::make(std::move(mb), result);
-    return error_code::success();
+    return error_code();
   }
 };
 }
index 80f50e8..fe7bf7f 100644 (file)
@@ -73,11 +73,11 @@ public:
     llvm::raw_fd_ostream out(outPath.data(), errorInfo,
                              llvm::sys::fs::F_None);
     if (!errorInfo.empty())
-      return error_code::success(); // FIXME
+      return error_code(); // FIXME
 
     this->write(out);
 
-    return error_code::success();
+    return error_code();
   }
 
   virtual ~Writer() {
index d924c42..d8f7758 100644 (file)
@@ -78,7 +78,7 @@ public:
 
   error_code
   parseAllMembers(std::vector<std::unique_ptr<File>> &result) const override {
-    return error_code::success();
+    return error_code();
   }
 
 private:
index 19a5509..43dee44 100644 (file)
@@ -318,7 +318,7 @@ error_code FileCOFF::parse() {
     return ec;
   if (error_code ec = maybeCreateSXDataAtoms())
     return ec;
-  return error_code::success();
+  return error_code();
 }
 
 /// Iterate over the symbol table to retrieve all symbols.
@@ -364,7 +364,7 @@ error_code FileCOFF::readSymbolTable(vector<const coff_symbol *> &result) {
  next:
       i += sym->NumberOfAuxSymbols;
   }
-  return error_code::success();
+  return error_code();
 }
 
 /// Create atoms for the absolute symbols.
@@ -436,7 +436,7 @@ FileCOFF::createUndefinedAtoms(const SymbolVectorT &symbols,
     result.push_back(atom);
     _symbolAtom[sym] = atom;
   }
-  return error_code::success();
+  return error_code();
 }
 
 /// Create atoms for the defined symbols. This pass is a bit complicated than
@@ -526,7 +526,7 @@ error_code FileCOFF::createDefinedSymbols(const SymbolVectorT &symbols,
   if (error_code ec = AtomizeDefinedSymbols(definedSymbols, result))
     return ec;
 
-  return error_code::success();
+  return error_code();
 }
 
 // Cache the COMDAT attributes, which indicate whether the symbols in the
@@ -567,7 +567,7 @@ error_code FileCOFF::cacheSectionAttributes() {
     if (!_merge.count(sec))
       _merge[sec] = DefinedAtom::mergeNo;
   }
-  return error_code::success();
+  return error_code();
 }
 
 /// Atomize \p symbols and append the results to \p atoms. The symbols are
@@ -603,7 +603,7 @@ FileCOFF::AtomizeDefinedSymbolsInSection(const coff_section *section,
       atoms.push_back(atom);
       _symbolAtom[sym] = atom;
     }
-    return error_code::success();
+    return error_code();
   }
 
   ArrayRef<uint8_t> secData;
@@ -614,7 +614,7 @@ FileCOFF::AtomizeDefinedSymbolsInSection(const coff_section *section,
   // a part of the output image. That's what the COFF spec says.
   if (section->Characteristics & llvm::COFF::IMAGE_SCN_LNK_INFO ||
       section->Characteristics & llvm::COFF::IMAGE_SCN_LNK_REMOVE)
-    return error_code::success();
+    return error_code();
 
   // Supporting debug info needs more work than just linking and combining
   // .debug sections. We don't support it yet. Let's discard .debug sections at
@@ -622,7 +622,7 @@ FileCOFF::AtomizeDefinedSymbolsInSection(const coff_section *section,
   // blobs that nobody would understand.
   if ((section->Characteristics & llvm::COFF::IMAGE_SCN_MEM_DISCARDABLE) &&
       (sectionName == ".debug" || sectionName.startswith(".debug$"))) {
-    return error_code::success();
+    return error_code();
   }
 
   DefinedAtom::ContentType type = getContentType(section);
@@ -637,7 +637,7 @@ FileCOFF::AtomizeDefinedSymbolsInSection(const coff_section *section,
         perms, _merge[section], data, _ordinal++);
     atoms.push_back(atom);
     _definedAtomLocations[section][0].push_back(atom);
-    return error_code::success();
+    return error_code();
   }
 
   // Create an unnamed atom if the first atom isn't at the start of the
@@ -669,7 +669,7 @@ FileCOFF::AtomizeDefinedSymbolsInSection(const coff_section *section,
   // Finally, set alignment to the first atom so that the section contents
   // will be aligned as specified by the object section header.
   _definedAtomLocations[section][0][0]->setAlignment(getAlignment(section));
-  return error_code::success();
+  return error_code();
 }
 
 error_code
@@ -692,7 +692,7 @@ FileCOFF::AtomizeDefinedSymbols(SectionToSymbolsT &definedSymbols,
       definedAtoms.push_back(atom);
     }
   }
-  return error_code::success();
+  return error_code();
 }
 
 /// Find the atom that is at \p targetAddress in \p section.
@@ -708,7 +708,7 @@ error_code FileCOFF::findAtomAt(const coff_section *section,
         targetAddress < atomAddress + atom->size()) {
       result = atom;
       offsetInAtom = targetAddress - atomAddress;
-      return error_code::success();
+      return error_code();
     }
   }
   // Relocation target is out of range
@@ -723,7 +723,7 @@ error_code FileCOFF::getAtomBySymbolIndex(uint32_t index, Atom *&ret) {
     return ec;
   ret = _symbolAtom[symbol];
   assert(ret);
-  return error_code::success();
+  return error_code();
 }
 
 /// Add relocation information to an atom based on \p rel. \p rel is an
@@ -751,7 +751,7 @@ FileCOFF::addRelocationReference(const coff_relocation *rel,
       new COFFReference(targetAtom, offsetInAtom, rel->Type,
                         Reference::KindNamespace::COFF,
                         _referenceArch)));
-  return error_code::success();
+  return error_code();
 }
 
 // Read section contents.
@@ -761,10 +761,10 @@ error_code FileCOFF::getSectionContents(StringRef sectionName,
   if (error_code ec = findSection(sectionName, section))
     return ec;
   if (!section)
-    return error_code::success();
+    return error_code();
   if (error_code ec = _obj->getSectionContents(section, result))
     return ec;
-  return error_code::success();
+  return error_code();
 }
 
 /// Returns the target machine type of the current object file.
@@ -775,13 +775,13 @@ error_code FileCOFF::getReferenceArch(Reference::KindArch &result) {
   switch (header->Machine) {
   case llvm::COFF::IMAGE_FILE_MACHINE_I386:
     result = Reference::KindArch::x86;
-    return error_code::success();
+    return error_code();
   case llvm::COFF::IMAGE_FILE_MACHINE_AMD64:
     result = Reference::KindArch::x86_64;
-    return error_code::success();
+    return error_code();
   case llvm::COFF::IMAGE_FILE_MACHINE_UNKNOWN:
     result = Reference::KindArch::all;
-    return error_code::success();
+    return error_code();
   }
   llvm::errs() << "Unsupported machine type: " << header->Machine << "\n";
   return llvm::object::object_error::parse_failed;
@@ -806,7 +806,7 @@ error_code FileCOFF::addRelocationReferenceToAtoms() {
         return ec;
     }
   }
-  return error_code::success();
+  return error_code();
 }
 
 // Read .sxdata section if exists. .sxdata is a x86-only section that contains a
@@ -827,7 +827,7 @@ error_code FileCOFF::maybeCreateSXDataAtoms() {
   if (error_code ec = getSectionContents(".sxdata", sxdata))
     return ec;
   if (sxdata.empty())
-    return error_code::success();
+    return error_code();
 
   std::vector<uint8_t> atomContent =
       *new (_alloc) std::vector<uint8_t>((size_t)sxdata.size());
@@ -851,7 +851,7 @@ error_code FileCOFF::maybeCreateSXDataAtoms() {
   }
 
   _definedAtoms._atoms.push_back(atom);
-  return error_code::success();
+  return error_code();
 }
 
 /// Find a section by name.
@@ -863,12 +863,12 @@ error_code FileCOFF::findSection(StringRef name, const coff_section *&result) {
       return ec;
     if (sectionName == name) {
       result = section;
-      return error_code::success();
+      return error_code();
     }
   }
   // Section was not found, but it's not an error. This method returns
   // an error only when there's a read error.
-  return error_code::success();
+  return error_code();
 }
 
 // Convert ArrayRef<uint8_t> to std::string. The array contains a string which
@@ -931,7 +931,7 @@ public:
     if (error_code ec = file->parse())
       return ec;
     result.push_back(std::move(file));
-    return error_code::success();
+    return error_code();
   }
 
 private:
@@ -1048,7 +1048,7 @@ public:
     createAlternateNameAtoms(*file);
 
     result.push_back(std::move(file));
-    return error_code::success();
+    return error_code();
   }
 
 private:
@@ -1086,7 +1086,7 @@ private:
     if (!errorMessage.empty()) {
       llvm::errs() << "lld warning: " << errorMessage << "\n";
     }
-    return error_code::success();
+    return error_code();
   }
 
   AliasAtom *createAlias(FileCOFF &file, StringRef name,
index a336146..8b03faa 100644 (file)
@@ -211,7 +211,7 @@ public:
     if (type == llvm::COFF::IMPORT_CODE)
       addDefinedAtom(symbolName, dllName, dataAtom);
 
-    ec = error_code::success();
+    ec = error_code();
   }
 
   const atom_collection<DefinedAtom> &defined() const override {
@@ -307,7 +307,7 @@ public:
     if (ec)
       return ec;
     result.push_back(std::move(file));
-    return error_code::success();
+    return error_code();
   }
 };
 
@@ -331,7 +331,7 @@ error_code parseCOFFImportLibrary(const LinkingContext &targetInfo,
   if (ec)
     return ec;
   result.push_back(std::move(file));
-  return error_code::success();
+  return error_code();
 }
 
 } // end namespace pecoff
index dbc5087..90ec430 100644 (file)
@@ -663,7 +663,7 @@ template <> struct MappingTraits<const lld::File *> {
 
     virtual error_code
     parseAllMembers(std::vector<std::unique_ptr<File>> &result) const override {
-      return error_code::success();
+      return error_code();
     }
 
     StringRef               _path;
@@ -1308,7 +1308,7 @@ public:
     const lld::File *fileRef = &file;
     yout << fileRef;
 
-    return error_code::success();
+    return error_code();
   }
 
 private: