use CallSite::isCalle instead of slow getOperandNo
authorGabor Greif <ggreif@gmail.com>
Fri, 23 Jan 2009 21:17:04 +0000 (21:17 +0000)
committerGabor Greif <ggreif@gmail.com>
Fri, 23 Jan 2009 21:17:04 +0000 (21:17 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@62877 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/IPO/DeadArgumentElimination.cpp

index 9d7980c0b2b9009e30ab9af747d074cfe51a1ae7..dd5fce69fe15f7f397fae2966af7e2323189780e 100644 (file)
@@ -182,7 +182,7 @@ bool DAE::DeleteDeadVarargs(Function &Fn) {
     if (!TheCall) return false;   // Not a direct call site?
 
     // The addr of this function is passed to the call.
-    if (I.getOperandNo() != 0) return false;
+    if (!CS.isCallee(I)) return false;
   }
 
   // Okay, we know we can transform this function if safe.  Scan its body
@@ -438,13 +438,13 @@ void DAE::SurveyFunction(Function &F) {
   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) {
+    CallSite CS = CallSite::get(*I);
+    if (!CS.getInstruction() || !CS.isCallee(I)) {
       MarkLive(F);
       return;
     }
 
     // If this use is anything other than a call site, the function is alive.
-    CallSite CS = CallSite::get(*I);
     Instruction *TheCall = CS.getInstruction();
     if (!TheCall) {   // Not a direct call site?
       MarkLive(F);