Revert of r223763, in spirit.

r223763 was made to work around a temporary issue where a user of the
JIT was passing down a declaration (incorrectly). This shouldn't
occur, so assert rather than silently continue.

llvm-svn: 224277

diff --git a/llvm/lib/Linker/LinkModules.cpp b/llvm/lib/Linker/LinkModules.cpp
index bb59400..9c3f183 100644 (file)
--- a/llvm/lib/Linker/LinkModules.cpp
+++ b/llvm/lib/Linker/LinkModules.cpp
@@ -1533,8 +1533,7 @@ bool ModuleLinker::run() {
     LazilyLinkGlobalValues.pop_back();
 
     if (auto F = dyn_cast<Function>(SGV))
-      if (F->isDeclaration())
-        continue;
+      assert(!F->isDeclaration() && "users should not pass down decls");
     if (linkGlobalValueBody(*SGV))
       return true;
   }