I guess the patch is fine, then. Committing...<br><br><div class="gmail_quote">On Sat, Oct 23, 2010 at 10:22 PM, nicolas geoffray <span dir="ltr"><<a href="mailto:nicolas.geoffray@gmail.com">nicolas.geoffray@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi all,<div><br></div><div>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).</div>

<div><br></div><div>Because a MachineInfo object is temporary, I believe it is fine to add a field there. OK to apply?</div><div><br></div><div>Thanks,</div><div>Nicolas</div>
</blockquote></div><br>