[Orc] Use '= default' for move constructor/assignment as per dblaikie's review.
authorLang Hames <lhames@gmail.com>
Mon, 19 Oct 2015 22:49:18 +0000 (22:49 +0000)
committerLang Hames <lhames@gmail.com>
Mon, 19 Oct 2015 22:49:18 +0000 (22:49 +0000)
Thanks Dave!

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@250749 91177308-0d34-0410-b5e6-96231b3b80d8

include/llvm/ExecutionEngine/Orc/CompileOnDemandLayer.h

index bf649af..a72805d 100644 (file)
@@ -69,21 +69,12 @@ private:
 
     LogicalModuleResources() {}
 
-    LogicalModuleResources(LogicalModuleResources &&Other) {
-      SourceModule = std::move(Other.SourceModule);
-      StubsToClone = std::move(StubsToClone);
-      StubsMgr = std::move(StubsMgr);
-    }
-
     // Explicit move constructor to make MSVC happy.
-    LogicalModuleResources& operator=(LogicalModuleResources &&Other) {
-      SourceModule = std::move(Other.SourceModule);
-      StubsToClone = std::move(StubsToClone);
-      StubsMgr = std::move(StubsMgr);
-      return *this;
-    }
+    LogicalModuleResources(LogicalModuleResources &&Other) = default;
 
     // Explicit move assignment to make MSVC happy.
+    LogicalModuleResources& operator=(LogicalModuleResources &&Other) = default;
+
     JITSymbol findSymbol(StringRef Name, bool ExportedSymbolsOnly) {
       if (Name.endswith("$stub_ptr") && !ExportedSymbolsOnly) {
         assert(!ExportedSymbolsOnly && "Stubs are never exported");