From 551b8eeb5cc9252116933a3670e92ee9a568ec3d Mon Sep 17 00:00:00 2001 From: Owen Anderson Date: Tue, 19 Feb 2008 03:15:29 +0000 Subject: 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. llvm-svn: 47309 --- llvm/lib/Transforms/Scalar/GVN.cpp | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) (limited to 'llvm/lib') diff --git a/llvm/lib/Transforms/Scalar/GVN.cpp b/llvm/lib/Transforms/Scalar/GVN.cpp index f34ae69779e..4de0f3dc725 100644 --- a/llvm/lib/Transforms/Scalar/GVN.cpp +++ b/llvm/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(); MD.dropInstruction(C); -- cgit v1.2.3