diff options
author | Rafael Espindola <rafael.espindola@gmail.com> | 2017-09-21 23:13:36 +0000 |
---|---|---|
committer | Rafael Espindola <rafael.espindola@gmail.com> | 2017-09-21 23:13:36 +0000 |
commit | 25cbdf25a667999485d79735f1a9ea61faf79eee (patch) | |
tree | 1bd920b92779d9706db8679815f730863eb83f11 /llvm/lib/Object/COFFImportFile.cpp | |
parent | 3306fff625738ce2946770f411dd8341d9a182d6 (diff) | |
download | llvm-25cbdf25a667999485d79735f1a9ea61faf79eee.zip llvm-25cbdf25a667999485d79735f1a9ea61faf79eee.tar.gz llvm-25cbdf25a667999485d79735f1a9ea61faf79eee.tar.bz2 |
Convert the archive writer to use Error.
This found one place in lld that was not checking the error.
llvm-svn: 313937
Diffstat (limited to 'llvm/lib/Object/COFFImportFile.cpp')
-rw-r--r-- | llvm/lib/Object/COFFImportFile.cpp | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/llvm/lib/Object/COFFImportFile.cpp b/llvm/lib/Object/COFFImportFile.cpp index 5f4b890..acac1e6 100644 --- a/llvm/lib/Object/COFFImportFile.cpp +++ b/llvm/lib/Object/COFFImportFile.cpp @@ -558,9 +558,9 @@ NewArchiveMember ObjectFactory::createWeakExternal(StringRef Sym, return {MemoryBufferRef(StringRef(Buf, Buffer.size()), ImportName)}; } -std::error_code writeImportLibrary(StringRef ImportName, StringRef Path, - ArrayRef<COFFShortExport> Exports, - MachineTypes Machine, bool MakeWeakAliases) { +Error writeImportLibrary(StringRef ImportName, StringRef Path, + ArrayRef<COFFShortExport> Exports, + MachineTypes Machine, bool MakeWeakAliases) { std::vector<NewArchiveMember> Members; ObjectFactory OF(llvm::sys::path::filename(ImportName), Machine); @@ -596,9 +596,8 @@ std::error_code writeImportLibrary(StringRef ImportName, StringRef Path, ? SymbolName : replace(SymbolName, E.Name, E.ExtName); - if (!Name) { - return errorToErrorCode(Name.takeError()); - } + if (!Name) + return Name.takeError(); Members.push_back( OF.createShortImport(*Name, E.Ordinal, ImportType, NameType)); |