[llvm] r217181 - Fix use-after-move introduced in r217065 and caught in post-commit review by Alexey.

David Blaikie dblaikie at gmail.com
Thu Sep 4 11:37:32 PDT 2014


Author: dblaikie
Date: Thu Sep  4 13:37:31 2014
New Revision: 217181

URL: http://llvm.org/viewvc/llvm-project?rev=217181&view=rev
Log:
Fix use-after-move introduced in r217065 and caught in post-commit review by Alexey.

Modified:
    llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp

Modified: llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp?rev=217181&r1=217180&r2=217181&view=diff
==============================================================================
--- llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp (original)
+++ llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp Thu Sep  4 13:37:31 2014
@@ -104,9 +104,9 @@ void MCJIT::addObjectFile(std::unique_pt
   if (!LoadedObject || Dyld.hasError())
     report_fatal_error(Dyld.getErrorString());
 
-  LoadedObjects.push_back(std::move(LoadedObject));
-
   NotifyObjectEmitted(*LoadedObject);
+
+  LoadedObjects.push_back(std::move(LoadedObject));
 }
 
 void MCJIT::addObjectFile(object::OwningBinary<object::ObjectFile> Obj) {





More information about the llvm-commits mailing list