[llvm-commits] [llvm] r91626 - in /llvm/trunk: lib/ExecutionEngine/JIT/JIT.cpp lib/ExecutionEngine/JIT/JIT.h lib/ExecutionEngine/JIT/JITEmitter.cpp unittests/ExecutionEngine/JIT/JITTest.cpp

Jeffrey Yasskin jyasskin at google.com
Tue Dec 22 12:52:33 PST 2009


On Mon, Dec 21, 2009 at 9:38 PM, nicolas geoffray
<nicolas.geoffray at gmail.com> wrote:
> Hi Jeffrey,
>
>>
>> Ah, I see now. You have your own ModuleProvider, and it needs to know
>> precisely when the function is actually needed, right? Yeah, it sounds
>> like I need to roll back that part of this fix until PR5737 is fixed.
>>
>
> Yes, thanks!

How's the attached patch look? I discovered that one of the
isDeclaration()s was untested, so I added a test for it.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: revert-materialize.patch
Type: application/octet-stream
Size: 7885 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20091222/5986cf2c/attachment.obj>


More information about the llvm-commits mailing list