From 600135a049942dc1ab3170be2f993d1802ff234c Mon Sep 17 00:00:00 2001
From: Chris Lattner <sabre@nondot.org>
Date: Sun, 28 Apr 2002 16:04:26 +0000
Subject: [PATCH] The check to see if an external function was marked internal
 was not reachable!

llvm-svn: 2366
---
 lib/VMCore/Verifier.cpp | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/lib/VMCore/Verifier.cpp b/lib/VMCore/Verifier.cpp
index 691f1182c3d..cd2aa842c31 100644
--- a/lib/VMCore/Verifier.cpp
+++ b/lib/VMCore/Verifier.cpp
@@ -69,6 +69,11 @@ namespace {  // Anonymous namespace for class
     }
 
     bool doFinalization(Module *M) {
+      // Scan through, checking all of the external function's linkage now...
+      for (Module::iterator I = M->begin(), E = M->end(); I != E; ++I)
+        if ((*I)->isExternal() && (*I)->hasInternalLinkage())
+          CheckFailed("", "Function Declaration has Internal Linkage!", (*I));
+
       if (Broken) {
         cerr << "Broken module found, compilation aborted!\n";
         abort();
@@ -135,11 +140,8 @@ void Verifier::verifySymbolTable(SymbolTable *ST) {
 //
 void Verifier::visitFunction(Function *F) {
   if (F->isExternal()) return;
-  verifySymbolTable(F->getSymbolTable());
 
-  // Check linkage of function...
-  Assert1(!F->isExternal() || F->hasExternalLinkage(),
-          "Function cannot be an 'internal' 'declare'ation!", F);
+  verifySymbolTable(F->getSymbolTable());
 
   // Check function arguments...
   const FunctionType *FT = F->getFunctionType();