From 9eb948b46623977cd4de7d4156b3c3058a7ebed3 Mon Sep 17 00:00:00 2001 From: Owen Anderson Date: Mon, 18 Feb 2008 04:06:26 +0000 Subject: [PATCH] Duncan pointed out that we can fast fail here, because the sret parameter of a function must be the first parameter. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@47254 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/VMCore/Function.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/VMCore/Function.cpp b/lib/VMCore/Function.cpp index fe3f2fe770c..e7a7041d778 100644 --- a/lib/VMCore/Function.cpp +++ b/lib/VMCore/Function.cpp @@ -107,7 +107,8 @@ bool Argument::hasNoAliasAttr() const { /// it in its containing function. bool Argument::hasStructRetAttr() const { if (!isa(getType())) return false; - return getParent()->paramHasAttr(getArgNo()+1, ParamAttr::StructRet); + if (getArgNo()) return false; // StructRet param must be first param + return getParent()->paramHasAttr(1, ParamAttr::StructRet); }