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
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 @@
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 @@
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) {