[llvm-commits] [llvm-gcc-4.2] r74575 - /llvm-gcc-4.2/trunk/gcc/llvm-convert.cpp
Chris Lattner
clattner at apple.com
Tue Jun 30 18:10:39 PDT 2009
On Jun 30, 2009, at 5:51 PM, Dale Johannesen wrote:
>>
>> It's not that big of a deal, but please use SmallVector<int, 4> or
>> something instead of alloca. SmallVector works when "N" is more than
>> 4 by going to the heap so you won't run out of stack space for large
>> N, and is more portable.
>
> I am not a big lover of alloca, but it was used several places in this
> code before I started working on it so I don't think portability can
> be a problem, and the numbers involved cannot be big enough to run out
> of stack space.
Ok, works for me. Please don't use it in normal llvm code though,
llvm-gcc is fine.
-Chris
More information about the llvm-commits
mailing list