[cfe-commits] r104919 - /cfe/trunk/lib/CodeGen/CGBuiltin.cpp

[Chris Lattner]

Yay!  Is there a bugzilla about this?

On May 27, 2010, at 6:45 PM, Dan Gohman wrote:

> Author: djg
> Date: Thu May 27 20:45:35 2010
> New Revision: 104919
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=104919&view=rev
> Log:
> This cast is no longer needed; the FIXME is fixed.
> 
> Modified:
>    cfe/trunk/lib/CodeGen/CGBuiltin.cpp
> 
> Modified: cfe/trunk/lib/CodeGen/CGBuiltin.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGBuiltin.cpp?rev=104919&r1=104918&r2=104919&view=diff
> ==============================================================================
> --- cfe/trunk/lib/CodeGen/CGBuiltin.cpp (original)
> +++ cfe/trunk/lib/CodeGen/CGBuiltin.cpp Thu May 27 20:45:35 2010
> @@ -398,9 +398,7 @@
> 
>   case Builtin::BIalloca:
>   case Builtin::BI__builtin_alloca: {
> -    // FIXME: LLVM IR Should allow alloca with an i64 size!
>     Value *Size = EmitScalarExpr(E->getArg(0));
> -    Size = Builder.CreateIntCast(Size, llvm::Type::getInt32Ty(VMContext), false, "tmp");
>     return RValue::get(Builder.CreateAlloca(llvm::Type::getInt8Ty(VMContext), Size, "tmp"));
>   }
>   case Builtin::BIbzero:
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits