[llvm] r194859 - Fix a problem in MCJIT identifying the module containing a global variable.

Andrew Kaylor andrew.kaylor at intel.com
Fri Nov 15 14:10:21 PST 2013


Author: akaylor
Date: Fri Nov 15 16:10:21 2013
New Revision: 194859

URL: http://llvm.org/viewvc/llvm-project?rev=194859&view=rev
Log:
Fix a problem in MCJIT identifying the module containing a global variable.

Patch by Keno Fischer!


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=194859&r1=194858&r2=194859&view=diff
==============================================================================
--- llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp (original)
+++ llvm/trunk/lib/ExecutionEngine/MCJIT/MCJIT.cpp Fri Nov 15 16:10:21 2013
@@ -248,11 +248,11 @@ Module *MCJIT::findModuleForSymbol(const
        I != E; ++I) {
     Module *M = *I;
     Function *F = M->getFunction(Name);
-    if (F && !F->empty())
+    if (F && !F->isDeclaration())
       return M;
     if (!CheckFunctionsOnly) {
       GlobalVariable *G = M->getGlobalVariable(Name);
-      if (G)
+      if (G && !G->isDeclaration())
         return M;
       // FIXME: Do we need to worry about global aliases?
     }





More information about the llvm-commits mailing list