aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/Object/COFFImportFile.cpp
diff options
context:
space:
mode:
authorRafael Espindola <rafael.espindola@gmail.com>2017-09-21 23:13:36 +0000
committerRafael Espindola <rafael.espindola@gmail.com>2017-09-21 23:13:36 +0000
commit25cbdf25a667999485d79735f1a9ea61faf79eee (patch)
tree1bd920b92779d9706db8679815f730863eb83f11 /llvm/lib/Object/COFFImportFile.cpp
parent3306fff625738ce2946770f411dd8341d9a182d6 (diff)
downloadllvm-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.cpp11
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));