D/ldc
1
0
Fork 0
mirror of https://github.com/xomboverlord/ldc.git synced 2026-01-26 09:33:13 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

Increment the iterator before deleting redundant functions or globals in the StripExternals pass.

Browse Source
This commit is contained in:
Christian Kamm
2009-06-11 21:48:10 +02:00
parent bc7737a03b
commit 8bcc2d9b3f
1 changed files with 13 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
gen/passes/StripExternals.cpp
View File

@@ -54,36 +54,44 @@ ModulePass *createStripExternalsPass() { return new StripExternals(); }
bool StripExternals::runOnModule(Module &M) {
bool Changed = false;
for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I) {
for (Module::iterator I = M.begin(); I != M.end(); ) {
if (I->hasAvailableExternallyLinkage()) {
assert(!I->isDeclaration()&&"Declarations can't be available_externally");
Changed = true;
++NumFunctions;
if (I->use_empty()) {
DOUT << "Deleting function: " << *I;
I->eraseFromParent();
Module::iterator todelete = I;
++I;
todelete->eraseFromParent();
continue;
} else {
I->deleteBody();
DOUT << "Deleted function body: " << *I;
}
}
++I;
}
for (Module::global_iterator I = M.global_begin(), E = M.global_end();
I != E; ++I) {
for (Module::global_iterator I = M.global_begin();
I != M.global_end(); ) {
if (I->hasAvailableExternallyLinkage()) {
assert(!I->isDeclaration()&&"Declarations can't be available_externally");
Changed = true;
++NumVariables;
if (I->use_empty()) {
DOUT << "Deleting global: " << *I;
I->eraseFromParent();
Module::global_iterator todelete = I;
++I;
todelete->eraseFromParent();
continue;
} else {
I->setInitializer(0);
I->setLinkage(GlobalValue::ExternalLinkage);
DOUT << "Deleted initializer: " << *I;
}
}
++I;
}
return Changed;