[llvm-commits] Move getPointerToNamedFunction() from JIT/MCJIT to JITMemoryManager.

[Danil Malyshev]

Hello everyone,


The move_getPointerToNamedFunction-01.patch has causing failures in the LazyLoadedJITTest.MaterializableAvailableExternallyFunctionIsntCompiled and MultiJitTest.JitPool unit tests.
This tests uses JIT.getPointerToNamedFunctions() for JIT with NULL memory manager, so call JMM->getPointerToNamedFunctions() raise exception.
This patch move the implementation of getPointerToNamedFunctions from JIT to defaultJMM, so I think, the best way is change this tests to use JIT with default memory manager.
Please review attached move_getPointerToNamedFunction-02.patch with these changes.
For easily looking I also attached the changes.diff. It's contains only changes linked with the problem.

Tested in Ubuntu 64 and Mac OS 64.


Regards,
Danil


________________________________
From: Danil Malyshev
Sent: Wednesday, March 21, 2012 10:41 PM
To: 'Jim Grosbach'
Cc: llvm-commits at cs.uiuc.edu
Subject: RE: [llvm-commits] Move getPointerToNamedFunction() from JIT/MCJIT to JITMemoryManager.

Hello,


Committed at r153205, thank you.



Regards,
Danil

________________________________
From: Jim Grosbach [mailto:grosbach at apple.com]
Sent: Wednesday, March 21, 2012 1:46 AM
To: Danil Malyshev
Cc: llvm-commits at cs.uiuc.edu
Subject: Re: [llvm-commits] Move getPointerToNamedFunction() from JIT/MCJIT to JITMemoryManager.

LGTM. Thanks!
On Mar 20, 2012, at 11:48 AM, Danil Malyshev <dmalyshev at accesssoftek.com<mailto:dmalyshev at accesssoftek.com>> wrote:

Hello everyone,

Based on this discussion: http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20120305/138477.html
Please find attached the patch for review.
This patch doing following:
1. Declares a virtual function getPointerToNamedFunction() in JITMemoryManager
2. Moves the implementation of getPointerToNamedFunction() form JIT/MCJIT to DefaultJITMemoryManager.


Regards,
Danil
<move_getPointerToNamedFunction-01.patch>_______________________________________________
llvm-commits mailing list
llvm-commits at cs.uiuc.edu<mailto:llvm-commits at cs.uiuc.edu>
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120322/6f33a022/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: move_getPointerToNamedFunction-02.patch
Type: application/octet-stream
Size: 17140 bytes
Desc: move_getPointerToNamedFunction-02.patch
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120322/6f33a022/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: changes.diff
Type: application/octet-stream
Size: 2749 bytes
Desc: changes.diff
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120322/6f33a022/attachment-0001.obj>