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

Add an obvious missing safety check to DAE::RemoveDeadArgumentsFromCallers.

Browse Source 
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@126720 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Eli Friedman 2011-03-01 00:33:47 +00:00
parent d3e768ecab
commit f291ab2fba
2 changed files with 14 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -294,7 +294,7 @@ bool DAE::DeleteDeadVarargs(Function &Fn) {
/// instead. /// instead.
bool DAE::RemoveDeadArgumentsFromCallers(Function &Fn) bool DAE::RemoveDeadArgumentsFromCallers(Function &Fn)
{ {
if (Fn.isDeclaration()) if (Fn.isDeclaration() || Fn.mayBeOverridden())
return false; return false;
// Functions with local linkage should already have been handled. // Functions with local linkage should already have been handled.

13
test/Transforms/DeadArgElim/deadexternal.ll
View File

@ -37,3 +37,16 @@ entry:
call void @f(i32 %tmp) call void @f(i32 %tmp)
ret void ret void
} }
; Check that callers are not transformed for weak definitions.
define weak i32 @weak_f(i32 %x) nounwind {
entry:
ret i32 0
}
define void @weak_f_caller() nounwind {
entry:
; CHECK: call i32 @weak_f(i32 10)
%call = tail call i32 @weak_f(i32 10)
ret void
}