On Apr 28, 2011, at 11:16 AM, Stuart Hastings wrote:

> Author: stuart
> Date: Thu Apr 28 13:16:06 2011
> New Revision: 130417
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=130417&view=rev
> Log:
> Replace SmallVector with an array, as suggested by Frits van Bommel.  
> rdar://problem/7662569

Thanks Stuart,

> +  const llvm::Type* LLVMField[1] = { llvm::ArrayType::get(ElemTy, SizeRegs) 
> };

You don't actually need an array here (you can pass in a scalar instead). 

> +  const llvm::Type* STy = llvm::StructType::get(getVMContext(), LLVMField,
> +                                                true);
> +  return ABIArgInfo::getDirect(STy);

Please use "Type *STy" instead of "Type* STy".

-Chris

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to