<div dir="ltr">On Mon, Jul 29, 2013 at 5:44 AM, Duncan Sands <span dir="ltr"><<a href="mailto:duncan.sands@gmail.com" target="_blank">duncan.sands@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Hi Peter,<br>
<br>
On 26/07/13 22:57, Peter Collingbourne wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Ping?<br>
</blockquote>
<br>
I think requiring the prefix data to start with a bunch of machine specific<br>
magic bytes is really horrible.  Why can't codegen insert the magic code<br>
sequence automatically?  I don't think this patch should go in as it is.<br></blockquote><div><br></div><div>The user will have to know the size of the magic code sequence somehow.  Also, they might not want it, if they always adjust symbols with bitcast and GEP when computing the address of a function.</div>
<div><br></div><div>What if Peter implemented symbol_offset first, and then committed this, with the intention that they be used together?</div><div><a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2013-April/061511.html">http://lists.cs.uiuc.edu/pipermail/llvmdev/2013-April/061511.html</a></div>
<div><br></div></div></div></div>