From: Chris Lattner <sabre@nondot.org>
Date: Sun, 28 Apr 2002 05:48:34 +0000 (+0000)
Subject: Be careful not to make "external" function internal
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=100d6daca4e6b762735b32bb2a00ee32945e2a82;p=oota-llvm.git

Be careful not to make "external" function internal


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

diff --git a/lib/Transforms/IPO/Internalize.cpp b/lib/Transforms/IPO/Internalize.cpp
index aadde7c92cf..8bb1a9c111c 100644
--- a/lib/Transforms/IPO/Internalize.cpp
+++ b/lib/Transforms/IPO/Internalize.cpp
@@ -15,7 +15,7 @@ class InternalizePass : public Pass {
   virtual bool run(Module *M) {
     bool FoundMain = false;   // Look for a function named main...
     for (Module::iterator I = M->begin(), E = M->end(); I != E; ++I)
-      if ((*I)->getName() == "main") {
+      if ((*I)->getName() == "main" && !(*I)->isExternal()) {
         FoundMain = true;
         break;
       }
@@ -26,7 +26,8 @@ class InternalizePass : public Pass {
 
     // Found a main function, mark all functions not named main as internal.
     for (Module::iterator I = M->begin(), E = M->end(); I != E; ++I)
-      if ((*I)->getName() != "main")  // Leave the main function external
+      if ((*I)->getName() != "main" &&  // Leave the main function external
+          !(*I)->isExternal())          // Function must be defined here
         (*I)->setInternalLinkage(Changed = true);
 
     return Changed;