[PATCH] D94218: Fix two pessimizing moves.

Christian Sigg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 8 02:44:57 PST 2021


This revision was automatically updated to reflect the committed changes.
Closed by commit rG2fe625e9abd0: Fix two pessimizing moves. (authored by csigg).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94218/new/

https://reviews.llvm.org/D94218

Files:
  llvm/include/llvm/Support/TaskQueue.h
  mlir/include/mlir/Parser.h


Index: mlir/include/mlir/Parser.h
===================================================================
--- mlir/include/mlir/Parser.h
+++ mlir/include/mlir/Parser.h
@@ -67,7 +67,7 @@
   // contain the operations inside of it.
   if (failed(op.verify()))
     return OwningOpRef<ContainerOpT>();
-  return std::move(opRef);
+  return opRef;
 }
 } // end namespace detail
 
Index: llvm/include/llvm/Support/TaskQueue.h
===================================================================
--- llvm/include/llvm/Support/TaskQueue.h
+++ llvm/include/llvm/Support/TaskQueue.h
@@ -98,7 +98,7 @@
         IsTaskInFlight = true;
       }
     }
-    return std::move(F);
+    return F;
   }
 
 private:


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D94218.315324.patch
Type: text/x-patch
Size: 692 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210108/a6a18bf7/attachment.bin>


More information about the llvm-commits mailing list