6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2024-12-14 11:32:34 +00:00
Code Issues Projects Releases Wiki Activity

Don't let DeadArgumentElimination attempt to update callers when the return

Browse Source 
type wasn't changed.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@52538 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Matthijs Kooijman 2008-06-20 15:25:43 +00:00
parent 03d18569cb
commit df0891d2ff
1 changed files with 12 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
lib/Transforms/IPO/DeadArgumentElimination.cpp
View File

@ -743,11 +743,18 @@ bool DAE::RemoveDeadStuffFromFunction(Function *F) {
Args.clear();
if (!Call->use_empty()) {
if (New->getType() == Type::VoidTy)
// Our return value was unused, replace by null for now, uses will get
// removed later on
if (New->getType() == Call->getType()) {
// Return type not changed? Just replace users then
Call->replaceAllUsesWith(New);
New->takeName(Call);
} else if (New->getType() == Type::VoidTy) {
// Our return value has uses, but they will get removed later on.
// Replace by null for now.
Call->replaceAllUsesWith(Constant::getNullValue(Call->getType()));
else if (isa<StructType>(RetTy)) {
} else {
assert(isa<StructType>(RetTy) && "Return type changed, but not into a"
"void. The old return type must have"
"been a struct!");
// The original return value was a struct, update all uses (which are
// all extractvalue instructions).
for (Value::use_iterator I = Call->use_begin(), E = Call->use_end();
@ -781,11 +788,7 @@ bool DAE::RemoveDeadStuffFromFunction(Function *F) {
}
}
New->takeName(Call);
} else {
// The original function had a single return value
Call->replaceAllUsesWith(New);
New->takeName(Call);
}
}
}
// Finally, remove the old call from the program, reducing the use-count of