[llvm-commits] [PATCH][MCJIT] Removing DefaultJITMemoryManager from MCJITMemoryManager

[Danil Malyshev]

Hi everyone,


Please review attached the patch.
The MCJITMemoryManager shouldn't use the DefaultJITMemoryManager by following reason:
1. It does not have any methods for invalidating instruction cache, so attempt to execute the code might be fail in ARM platforms.
2. The repeated call of DefaultJITMemoryManager.allocateDataSection() fails.
But the MCJITMemoryManager misleading because his constructor create the DefaultJITMemoryManager if jmm parameter is empty. This patch fixed it.


Regards,
Danil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120725/6b238041/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: remove_DefaultJMM_from_MCJTMM-01.patch
Type: application/octet-stream
Size: 685 bytes
Desc: remove_DefaultJMM_from_MCJTMM-01.patch
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120725/6b238041/attachment.obj>