[PATCH] IntelJITEventListener updates to fix breaks by recent changes to EngineBuilder and DIContext

Rafael Ávila de Espíndola rafael.espindola at gmail.com
Wed Aug 20 09:55:58 PDT 2014


================
Comment at: tools/llvm-jitlistener/llvm-jitlistener.cpp:148
@@ -147,3 +147,3 @@
     std::string Error;
-    TheJIT.reset(EngineBuilder(TheModule)
+    TheJIT.reset(EngineBuilder(std::unique_ptr<Module>(TheModule))
       .setEngineKind(EngineKind::JIT)
----------------
This looks odd.

It seems TheModule is only used in InitEE. Could you make it a local variable:

std::unique_ptr<Module> TheModule( ParseIRFile(IRFile, Err, Context));

and std::move it here?

http://reviews.llvm.org/D4975






More information about the llvm-commits mailing list