<div dir="ltr">Belatedly adding my comments:<div><br></div><div>Eric's right (and knows far more about ELF than me). MCJIT should honor SHF_ALLOC. If it doesn't, please file a bug.</div><div><br></div><div>Cheers,</div>
<div>Lang.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 21, 2014 at 3:05 PM, Philip Reames <span dir="ltr"><<a href="mailto:listmail@philipreames.com" target="_blank">listmail@philipreames.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
On 07/21/2014 02:52 PM, Eric Christopher wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Mon, Jul 21, 2014 at 10:08 AM, Philip Reames<br>
<<a href="mailto:listmail@philipreames.com" target="_blank">listmail@philipreames.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
================<br>
Comment at: lib/MC/MCObjectFileInfo.cpp:<u></u>590<br>
@@ -585,1 +589,3 @@<br>
+                       SectionKind::getReadOnly());<br>
+<br>
  }<br>
----------------<br>
Juergen Ributzka wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I am not familiar with the ELF sections, but wouldn't Metadata be a better for for this section. I guess it doesn't work with it, because the JIT won't ask the client to allocate it?<br>
<br>
Lang knows more about this part of the JIT.<br>
</blockquote>
Worth noting:<br>
1) The COFF section uses getMetadata<br>
2) getMetadata appears to work in practice<br>
<br>
I'm happy to make the change, but don't feel like I understand the implications.  Reading through the documentation, I'm unclear on the semantics.  Lang, can you clarify?<br>
<br>
</blockquote>
Philip: I'm not sure what Lang can clarify past my comments a few days<br>
ago. Was anything in there unclear? (Other than my question about<br>
whether or not the JIT will allocate anything with SHF_ALLOC - if it<br>
doesn't we should fix the JIT).<br>
</blockquote></div></div>
I had managed to miss your comment.  Sorry.<br>
<br>
I will change to using getMetadata and update the diff.<span class="HOEnZb"><font color="#888888"><br>
<br>
Philip<br>
</font></span></blockquote></div><br></div>