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

[Nick Lewycky]

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.

Nick