Remove redundant move in return statement (#90546)

This pull request removes unnecessary move in the return statement to
suppress compilation warnings.

Co-authored-by: Xiaolei Shi <[email protected]>

Author: xiaoleis-nv

llvm/include/llvm/ExecutionEngine/Orc/IndirectionUtils.h

@@ -77,7 +77,7 @@ class TrampolinePool {
     std::lock_guard<std::mutex> Lock(TPMutex);
     if (AvailableTrampolines.empty()) {
       if (auto Err = grow())
-        return std::move(Err);
+        return Err;
     }
     assert(!AvailableTrampolines.empty() && "Failed to grow trampoline pool");
     auto TrampolineAddr = AvailableTrampolines.back();

llvm/include/llvm/ExecutionEngine/Orc/LLJIT.h

@@ -488,18 +488,18 @@ class LLJITBuilderSetters {
   /// Create an instance of the JIT.
   Expected<std::unique_ptr<JITType>> create() {
     if (auto Err = impl().prepareForConstruction())
-      return std::move(Err);
+      return Err;
 
     Error Err = Error::success();
     std::unique_ptr<JITType> J(new JITType(impl(), Err));
     if (Err)
-      return std::move(Err);
+      return Err;
 
     if (impl().NotifyCreated)
       if (Error Err = impl().NotifyCreated(*J))
-        return std::move(Err);
+        return Err;
 
-    return std::move(J);
+    return J;
   }
 
 protected: