llvm dev - Oct 2010 - [LLVMdev] [PATCH] Add a GCModuleInfo field to MachineFunction

Hi all,

This patch adds a GCModuleInfo field to the MachineFunction class, to
facilitate the retrieval of a potential GCFunctionInfo attached to the
function. The motivation for the change is to get access in a proper way to
the GC info when generating the function with the JIT. Currently, retrieving
this information requires a lot of hacks (eg VMKit's code
lib/J3/Compiler/JavaJITCompiler.cpp).

Because a MachineInfo object is temporary, I believe it is fine to add a
field there. OK to apply?

Thanks,
Nicolas
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20101023/2c5d9fa2/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: gc.patch
Type: application/octet-stream
Size: 2911 bytes
Desc: not available
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20101023/2c5d9fa2/attachment.obj>

I guess the patch is fine, then. Committing...

On Sat, Oct 23, 2010 at 10:22 PM, nicolas geoffray <
nicolas.geoffray at gmail.com> wrote:
> Hi all,
>
> This patch adds a GCModuleInfo field to the MachineFunction class, to
> facilitate the retrieval of a potential GCFunctionInfo attached to the
> function. The motivation for the change is to get access in a proper way to
> the GC info when generating the function with the JIT. Currently,
retrieving
> this information requires a lot of hacks (eg VMKit's code
> lib/J3/Compiler/JavaJITCompiler.cpp).
>
> Because a MachineInfo object is temporary, I believe it is fine to add a
> field there. OK to apply?
>
> Thanks,
> Nicolas
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20101031/8dd8e681/attachment.html>