replica
/
hanchenye-llvm-project
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Remove the code from Function::dropAllReferences which replaced 

uses of the function's blocks with undef. This code isn't needed,
because BasicBlock's destructor handles such uses. Also, undef isn't
correct, since blockaddresses may still be used for comparisons
with null.

llvm-svn: 121170
This commit is contained in:
Dan Gohman 2010-12-07 19:56:51 +00:00
parent d43384f608
commit f844b3b227
1 changed files with 4 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
llvm/lib/VMCore/Function.cpp
View File

@ -227,19 +227,10 @@ void Function::dropAllReferences() {
for (iterator I = begin(), E = end(); I != E; ++I)
I->dropAllReferences();
// Delete all basic blocks.
while (!BasicBlocks.empty()) {
// If there is still a reference to the block, it must be a 'blockaddress'
// constant pointing to it. Just replace the BlockAddress with undef.
BasicBlock *BB = BasicBlocks.begin();
if (!BB->use_empty()) {
BlockAddress *BA = cast<BlockAddress>(BB->use_back());
BA->replaceAllUsesWith(UndefValue::get(BA->getType()));
BA->destroyConstant();
}
BB->eraseFromParent();
}
// Delete all basic blocks. They are now unused, except possibly by
// blockaddresses, but BasicBlock's destructor takes care of those.
while (!BasicBlocks.empty())
BasicBlocks.begin()->eraseFromParent();
}
void Function::addAttribute(unsigned i, Attributes attr) {