commit 6d3425e87a899e4796ac24ebe0b90e9d785d23c3
author Chris Lattner <sabre@nondot.org> Wed Oct 22 04:43:18 2003 +0000
committer Chris Lattner <sabre@nondot.org> Wed Oct 22 04:43:18 2003 +0000
tree 174c9b953f022212eceeccf416c544b1c115fbe9
parent 12b3593dea960e6aba896a90600636226a9ec5a1

Update the 'used' flag correctly

llvm-svn: 9366

llvm/lib/Transforms/IPO/FunctionResolution.cpp

diff --git a/llvm/lib/Transforms/IPO/FunctionResolution.cpp b/llvm/lib/Transforms/IPO/FunctionResolution.cpp
index 844bc00..b8af502 100644
--- a/llvm/lib/Transforms/IPO/FunctionResolution.cpp
+++ b/llvm/lib/Transforms/IPO/FunctionResolution.cpp
@@ -245,24 +245,25 @@
   // algorithm here to avoid problems with iterators getting invalidated if we
   // did a one pass scheme.
   //
+  bool Changed = false;
   for (Module::iterator I = M.begin(), E = M.end(); I != E; ) {
     Function *F = I++;
-    if (F->use_empty() && F->isExternal())
+    if (F->use_empty() && F->isExternal()) {
       M.getFunctionList().erase(F);
-    else if (!F->hasInternalLinkage() && !F->getName().empty())
+      Changed = true;
+    } else if (!F->hasInternalLinkage() && !F->getName().empty())
       Globals[F->getName()].push_back(F);
   }
 
   for (Module::giterator I = M.gbegin(), E = M.gend(); I != E; ) {
     GlobalVariable *GV = I++;
-    if (GV->use_empty() && GV->isExternal())
+    if (GV->use_empty() && GV->isExternal()) {
       M.getGlobalList().erase(GV);
-    else if (!GV->hasInternalLinkage() && !GV->getName().empty())
+      Changed = true;
+    } else if (!GV->hasInternalLinkage() && !GV->getName().empty())
       Globals[GV->getName()].push_back(GV);
   }
 
-  bool Changed = false;
-
   TargetData &TD = getAnalysis<TargetData>();
 
   // Now we have a list of all functions with a particular name.  If there is