[llvm-commits] [llvm] r82100 - /llvm/trunk/lib/ExecutionEngine/Interpreter/ExternalFunctions.cpp

[Chris Lattner]

On Sep 18, 2009, at 9:47 AM, Nick Lewycky wrote:

> Daniel Dunbar wrote:
>> On Thu, Sep 17, 2009 at 9:46 AM, Chris Lattner <clattner at apple.com>  
>> wrote:
>>> On Sep 16, 2009, at 5:14 PM, Daniel Dunbar wrote:
>>>
>>>> Author: ddunbar
>>>> Date: Wed Sep 16 19:14:44 2009
>>>> New Revision: 82100
>>>>
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=82100&view=rev
>>>> Log:
>>>> Some platforms may need malloc.h for alloca.
>>> this isn't portable, can the code use smallvector instead of alloca?
>> Perhaps not, but it is incrementally better than not compiling. This
>> is just a response to a user report on IRC, it looks like the alloca
>> went in with the FFI patch. CCing Nick.
>
> I switched the alloca's to SmallVector and it looks like everything  
> still works.

Thanks!