Revert "Remove redundant "std::move"s in return statements"

The build failed with

  error: call to deleted constructor of 'llvm::Error'

errors.

This reverts commit 1c2241a7936bf85aa68aef94bd40c3ba77d8ddf2.

1 files changed, 4 insertions, 4 deletions

diff --git a/llvm/lib/Object/Archive.cpp b/llvm/lib/Object/Archive.cpp
index 4f7cea8..17d5eec 100644
--- a/llvm/lib/Object/Archive.cpp
+++ b/llvm/lib/Object/Archive.cpp
@@ -481,7 +481,7 @@ Expected<Archive::Child> Archive::Child::getNext() const {
   Error Err = Error::success();
   Child Ret(Parent, NextLoc, &Err);
   if (Err)
-    return Err;
+    return std::move(Err);
   return Ret;
 }
 
@@ -531,8 +531,8 @@ Expected<std::unique_ptr<Archive>> Archive::create(MemoryBufferRef Source) {
   Error Err = Error::success();
   std::unique_ptr<Archive> Ret(new Archive(Source, Err));
   if (Err)
-    return Err;
-  return Ret;
+    return std::move(Err);
+  return std::move(Ret);
 }
 
 void Archive::setFirstRegular(const Child &C) {
@@ -852,7 +852,7 @@ Expected<Archive::Child> Archive::Symbol::getMember() const {
   Error Err = Error::success();
   Child C(Parent, Loc, &Err);
   if (Err)
-    return Err;
+    return std::move(Err);
   return C;
 }