From: Owen Anderson <resistor@mac.com>
Date: Tue, 19 Feb 2008 03:15:29 +0000 (+0000)
Subject: An sret parameter is required to be the first parameter, so there's no need to loop
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=c0755a6f984f17c269c2a7a4b48c74d201403c9b;p=oota-llvm.git

An sret parameter is required to be the first parameter, so there's no need to loop
over all the parameters of the callee looking for it.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@47309 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/Transforms/Scalar/GVN.cpp b/lib/Transforms/Scalar/GVN.cpp
index f34ae69779e..4de0f3dc725 100644
--- a/lib/Transforms/Scalar/GVN.cpp
+++ b/lib/Transforms/Scalar/GVN.cpp
@@ -1089,15 +1089,10 @@ bool GVN::performReturnSlotOptzn(MemCpyInst* cpy, CallInst* C,
   
   // If all checks passed, then we can perform the transformation
   CallSite CS = CallSite::get(C);
-  for (unsigned i = 0; i < CS.arg_size(); ++i) {
-    if (CS.paramHasAttr(i+1, ParamAttr::StructRet)) {
-      if (CS.getArgument(i)->getType() != cpyDest->getType())
-        return false;
+  if (CS.getArgument(0)->getType() != cpyDest->getType())
+    return false;
       
-      CS.setArgument(i, cpyDest);
-      break;
-    }
-  }
+  CS.setArgument(0, cpyDest);
   
   MemoryDependenceAnalysis& MD = getAnalysis<MemoryDependenceAnalysis>();
   MD.dropInstruction(C);