Remove dead code from this function and optimize. Update its corresponding

LangRef entry too.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@82890 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Nick Lewycky 2009-09-27 07:55:32 +00:00
parent 6776064d19
commit 513869453b
2 changed files with 10 additions and 24 deletions

View File

@ -1592,7 +1592,7 @@ Classifications</a> </div>
<h5>Syntax:</h5>
<pre>
&lt;returntype list&gt; (&lt;parameter list&gt;)
&lt;returntype&gt; (&lt;parameter list&gt;)
</pre>
<p>...where '<tt>&lt;parameter list&gt;</tt>' is a comma-separated list of type
@ -1600,8 +1600,8 @@ Classifications</a> </div>
which indicates that the function takes a variable number of arguments.
Variable argument functions can access their arguments with
the <a href="#int_varargs">variable argument handling intrinsic</a>
functions. '<tt>&lt;returntype list&gt;</tt>' is a comma-separated list of
<a href="#t_firstclass">first class</a> type specifiers.</p>
functions. '<tt>&lt;returntype&gt;</tt>' is a any type except
<a href="#t_label">label</a> and <a href="#t_metadata">metadata</a>.</p>
<h5>Examples:</h5>
<table class="layout">
@ -1626,8 +1626,8 @@ Classifications</a> </div>
</td>
</tr><tr class="layout">
<td class="left"><tt>{i32, i32} (i32)</tt></td>
<td class="left">A function taking an <tt>i32</tt>, returning two
<tt>i32</tt> values as an aggregate of type <tt>{ i32, i32 }</tt>
<td class="left">A function taking an <tt>i32</tt>, returning a
<a href="#t_struct">structure</a> containing two <tt>i32</tt> values
</td>
</tr>
</table>

View File

@ -365,25 +365,11 @@ const IntegerType *Type::getInt64Ty(LLVMContext &C) {
/// isValidReturnType - Return true if the specified type is valid as a return
/// type.
bool FunctionType::isValidReturnType(const Type *RetTy) {
if (RetTy->isFirstClassType()) {
if (const PointerType *PTy = dyn_cast<PointerType>(RetTy))
return PTy->getElementType()->getTypeID() != MetadataTyID;
return true;
}
if (RetTy->getTypeID() == VoidTyID || RetTy->getTypeID() == MetadataTyID ||
isa<OpaqueType>(RetTy))
return true;
// If this is a multiple return case, verify that each return is a first class
// value and that there is at least one value.
const StructType *SRetTy = dyn_cast<StructType>(RetTy);
if (SRetTy == 0 || SRetTy->getNumElements() == 0)
return false;
for (unsigned i = 0, e = SRetTy->getNumElements(); i != e; ++i)
if (!SRetTy->getElementType(i)->isFirstClassType())
return false;
return true;
if (const PointerType *PTy = dyn_cast<PointerType>(RetTy))
return PTy->getElementType()->getTypeID() != MetadataTyID;
return RetTy->getTypeID() != LabelTyID &&
RetTy->getTypeID() != MetadataTyID;
}
/// isValidArgumentType - Return true if the specified type is valid as an