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 > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
