Remove dead code. We are only lazy about functions with bodies.

llvm-svn: 223521

diff --git a/llvm/lib/Linker/LinkModules.cpp b/llvm/lib/Linker/LinkModules.cpp
index c7cf9f6900351f5e19b590c0d04f383332168c05..de62c3b94d05809a0075bf4b7e4900d8de1d988b 100644 (file)
--- a/llvm/lib/Linker/LinkModules.cpp
+++ b/llvm/lib/Linker/LinkModules.cpp
@@ -1518,9 +1518,6 @@ bool ModuleLinker::run() {
     Function *SF = LazilyLinkFunctions.back();
     LazilyLinkFunctions.pop_back();
 
-    if (!SF)
-      continue;
-
     Function *DF = cast<Function>(ValueMap[SF]);
     if (SF->hasPrefixData()) {
       // Link in the prefix data.
@@ -1532,11 +1529,8 @@ bool ModuleLinker::run() {
     if (std::error_code EC = SF->materialize())
       return emitError(EC.message());
 
-    // Skip if no body (function is external).
-    if (SF->isDeclaration())
-      continue;
-
     // Link in function body.
+    assert(!SF->isDeclaration());
     linkFunctionBody(DF, SF);
     SF->Dematerialize();
   }