diff options
author | Bob Wilson <bob.wilson@apple.com> | 2010-09-23 17:25:06 +0000 |
---|---|---|
committer | Bob Wilson <bob.wilson@apple.com> | 2010-09-23 17:25:06 +0000 |
commit | 3aecb15f0afb9c92ca1f6de33653e75fa1240a6b (patch) | |
tree | 0c704e738216485f9c4cb7176bd9da9bc4e0614d /llvm/lib/Transforms/IPO/ExtractGV.cpp | |
parent | d5f4130ffbb42dd1918ba05430523f00f14d8c96 (diff) | |
download | llvm-3aecb15f0afb9c92ca1f6de33653e75fa1240a6b.zip llvm-3aecb15f0afb9c92ca1f6de33653e75fa1240a6b.tar.gz llvm-3aecb15f0afb9c92ca1f6de33653e75fa1240a6b.tar.bz2 |
Fix llvm-extract so that it changes the linkage of all GlobalValues to
"external" even when doing lazy bitcode loading. This was broken because
a function that is not materialized fails the !isDeclaration() test.
llvm-svn: 114666
Diffstat (limited to 'llvm/lib/Transforms/IPO/ExtractGV.cpp')
-rw-r--r-- | llvm/lib/Transforms/IPO/ExtractGV.cpp | 30 |
1 files changed, 14 insertions, 16 deletions
diff --git a/llvm/lib/Transforms/IPO/ExtractGV.cpp b/llvm/lib/Transforms/IPO/ExtractGV.cpp index bb04169..9d432de 100644 --- a/llvm/lib/Transforms/IPO/ExtractGV.cpp +++ b/llvm/lib/Transforms/IPO/ExtractGV.cpp @@ -50,24 +50,22 @@ namespace { // Visit the GlobalVariables. for (Module::global_iterator I = M.global_begin(), E = M.global_end(); - I != E; ++I) - if (!I->isDeclaration()) { - if (I->hasLocalLinkage()) - I->setVisibility(GlobalValue::HiddenVisibility); - I->setLinkage(GlobalValue::ExternalLinkage); - if (deleteStuff == (bool)Named.count(I)) - I->setInitializer(0); - } + I != E; ++I) { + if (I->hasLocalLinkage()) + I->setVisibility(GlobalValue::HiddenVisibility); + I->setLinkage(GlobalValue::ExternalLinkage); + if (deleteStuff == (bool)Named.count(I) && !I->isDeclaration()) + I->setInitializer(0); + } // Visit the Functions. - for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I) - if (!I->isDeclaration()) { - if (I->hasLocalLinkage()) - I->setVisibility(GlobalValue::HiddenVisibility); - I->setLinkage(GlobalValue::ExternalLinkage); - if (deleteStuff == (bool)Named.count(I)) - I->deleteBody(); - } + for (Module::iterator I = M.begin(), E = M.end(); I != E; ++I) { + if (I->hasLocalLinkage()) + I->setVisibility(GlobalValue::HiddenVisibility); + I->setLinkage(GlobalValue::ExternalLinkage); + if (deleteStuff == (bool)Named.count(I) && !I->isDeclaration()) + I->deleteBody(); + } return true; } |