There is no reason to try to materialize the function from bytecode if it

already has been.  This may be a small speedup.


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

1 files changed, 12 insertions, 9 deletions

diff --git a/lib/ExecutionEngine/JIT/JIT.cpp b/lib/ExecutionEngine/JIT/JIT.cpp
index d9866c87ee..71f1dd0d6b 100644
--- a/lib/ExecutionEngine/JIT/JIT.cpp
+++ b/lib/ExecutionEngine/JIT/JIT.cpp
@@ -239,15 +239,18 @@ void *JIT::getPointerToFunction(Function *F) {
     return Addr;   // Check if function already code gen'd
 
   // Make sure we read in the function if it exists in this Module
-  try {
-    MP->materializeFunction(F);
-  } catch ( std::string& errmsg ) {
-    std::cerr << "Error reading bytecode file: " << errmsg << "\n";
-    abort();
-  } catch (...) {
-    std::cerr << "Error reading bytecode file!\n";
-    abort();
-  }
+  if (F->hasNotBeenReadFromBytecode()) 
+    try {
+      MP->materializeFunction(F);
+    } catch ( std::string& errmsg ) {
+      std::cerr << "Error reading function '" << F->getName()
+                << "' from bytecode file: " << errmsg << "\n";
+      abort();
+    } catch (...) {
+      std::cerr << "Error reading function '" << F->getName()
+                << "from bytecode file!\n";
+      abort();
+    }
 
   if (F->isExternal()) {
     void *Addr = getPointerToNamedFunction(F->getName());