[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