diff options
author | Matthijs Kooijman <matthijs@stdin.nl> | 2008-06-05 08:34:25 +0000 |
---|---|---|
committer | Matthijs Kooijman <matthijs@stdin.nl> | 2008-06-05 08:34:25 +0000 |
commit | 41335411a84aaf709e79b86b7dc109976940b772 (patch) | |
tree | 9fee8058f65c9124301e5004497d5b613819fb8a | |
parent | 045b3f7bf1f136f609d64e50b3737b6eccae1ed8 (diff) | |
download | external_llvm-41335411a84aaf709e79b86b7dc109976940b772.zip external_llvm-41335411a84aaf709e79b86b7dc109976940b772.tar.gz external_llvm-41335411a84aaf709e79b86b7dc109976940b772.tar.bz2 |
Use use_iterator::getOperandNo instead of CallSite::hasArgument to check if a
function is passed as an argument instead of called. Also do this check a bit
earlier.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@51990 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/Transforms/IPO/DeadArgumentElimination.cpp | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/lib/Transforms/IPO/DeadArgumentElimination.cpp b/lib/Transforms/IPO/DeadArgumentElimination.cpp index 2f075ba..604a148 100644 --- a/lib/Transforms/IPO/DeadArgumentElimination.cpp +++ b/lib/Transforms/IPO/DeadArgumentElimination.cpp @@ -302,6 +302,12 @@ void DAE::SurveyFunction(Function &F) { FunctionIntrinsicallyLive = true; else for (Value::use_iterator I = F.use_begin(), E = F.use_end(); I != E; ++I) { + // If the function is PASSED IN as an argument, its address has been taken + if (I.getOperandNo() != 0) { + FunctionIntrinsicallyLive = true; + break; + } + // If this use is anything other than a call site, the function is alive. CallSite CS = CallSite::get(*I); Instruction *TheCall = CS.getInstruction(); @@ -329,13 +335,6 @@ void DAE::SurveyFunction(Function &F) { RetValLiveness = Live; break; } - - // If the function is PASSED IN as an argument, its address has been taken - - if (CS.hasArgument(&F)) { - FunctionIntrinsicallyLive = true; - break; - } } if (FunctionIntrinsicallyLive) { |