[Immediate] Explicitly move error when returning

Zak Kent · Zak Kent · commit 514fc1e55fd5 · 2023-08-14T13:28:23.000-07:00
diff --git a/lib/Immediate/SwiftMaterializationUnit.cpp b/lib/Immediate/SwiftMaterializationUnit.cpp
@@ -97,7 +97,7 @@ SwiftJIT::~SwiftJIT() {
 
 llvm::Expected<int> SwiftJIT::runMain(llvm::ArrayRef<std::string> Args) {
   if (auto Err = J->initialize(J->getMainJITDylib())) {
-    return Err;
+    return std::move(Err);
   }
 
   auto MainSym = J->lookup("main");
@@ -113,7 +113,7 @@ llvm::Expected<int> SwiftJIT::runMain(llvm::ArrayRef<std::string> Args) {
 
   LLVM_DEBUG(llvm::dbgs() << "Running static destructors\n");
   if (auto Err = J->deinitialize(J->getMainJITDylib())) {
-    return Err;
+    return std::move(Err);
   }
 
   return Result;

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ SwiftJIT::~SwiftJIT() {`
`97`	`97`
`98`	`98`	`llvm::Expected<int> SwiftJIT::runMain(llvm::ArrayRef<std::string> Args) {`
`99`	`99`	`if (auto Err = J->initialize(J->getMainJITDylib())) {`
`100`		`- return Err;`
	`100`	`+ return std::move(Err);`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`auto MainSym = J->lookup("main");`
`@@ -113,7 +113,7 @@ llvm::Expected<int> SwiftJIT::runMain(llvm::ArrayRef<std::string> Args) {`
`113`	`113`
`114`	`114`	`LLVM_DEBUG(llvm::dbgs() << "Running static destructors\n");`
`115`	`115`	`if (auto Err = J->deinitialize(J->getMainJITDylib())) {`
`116`		`- return Err;`
	`116`	`+ return std::move(Err);`
`117`	`117`	`}`
`118`	`118`
`119`	`119`	`return Result;`