diff options
author | Chris Lattner <sabre@nondot.org> | 2007-02-18 22:10:34 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2007-02-18 22:10:34 +0000 |
commit | eb47391994497060793d8f91d1c0a94e39538c6d (patch) | |
tree | d4034723c3a413e6c0aaf9d7832b15a4ff45a75d | |
parent | 439ba1fefa6a129089b5e06541b9b9d47594bf55 (diff) | |
download | external_llvm-eb47391994497060793d8f91d1c0a94e39538c6d.zip external_llvm-eb47391994497060793d8f91d1c0a94e39538c6d.tar.gz external_llvm-eb47391994497060793d8f91d1c0a94e39538c6d.tar.bz2 |
simplify pass, delete dead gvar protos as well.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@34394 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Transforms/IPO/StripDeadPrototypes.cpp | 37 |
1 files changed, 21 insertions, 16 deletions
diff --git a/lib/Transforms/IPO/StripDeadPrototypes.cpp b/lib/Transforms/IPO/StripDeadPrototypes.cpp index 543c68e..d433064 100644 --- a/lib/Transforms/IPO/StripDeadPrototypes.cpp +++ b/lib/Transforms/IPO/StripDeadPrototypes.cpp @@ -12,13 +12,12 @@ // //===----------------------------------------------------------------------===// +#define DEBUG_TYPE "strip-dead-prototypes" #include "llvm/Transforms/IPO.h" #include "llvm/Pass.h" #include "llvm/Module.h" #include "llvm/ADT/Statistic.h" -#include "llvm/Support/Debug.h" #include "llvm/Support/Compiler.h" -#include <vector> using namespace llvm; STATISTIC(NumDeadPrototypes, "Number of dead prototypes removed"); @@ -37,24 +36,30 @@ RegisterPass<StripDeadPrototypesPass> X("strip-dead-prototypes", } // end anonymous namespace bool StripDeadPrototypesPass::runOnModule(Module &M) { - // Collect all the functions we want to erase - std::vector<Function*> FuncsToErase; - for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I) - if (I->isDeclaration() && // Function must be only a prototype - I->use_empty()) { // Function must not be used - FuncsToErase.push_back(&(*I)); + bool MadeChange = false; + + // Erase dead function prototypes. + for (Module::iterator I = M.begin(), E = M.end(); I != E; ) { + Function *F = I++; + // Function must be a prototype and unused. + if (F->isDeclaration() && F->use_empty()) { + F->eraseFromParent(); + ++NumDeadPrototypes; + MadeChange = true; } + } - // Erase the functions - for (std::vector<Function*>::iterator I = FuncsToErase.begin(), - E = FuncsToErase.end(); I != E; ++I ) - (*I)->eraseFromParent(); + // Erase dead function prototypes. + for (Module::global_iterator I = M.global_begin(), E = M.global_end(); + I != E; ) { + GlobalVariable *GV = I++; + // Global must be a prototype and unused. + if (GV->isDeclaration() && GV->use_empty()) + GV->eraseFromParent(); + } - // Increment the statistic - NumDeadPrototypes += FuncsToErase.size(); - // Return an indication of whether we changed anything or not. - return !FuncsToErase.empty(); + return MadeChange; } ModulePass *llvm::createStripDeadPrototypesPass() { |