<div dir="ltr">Rafael, thanks for fixing PR 20127. Committed this patch in r214520.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Aug 1, 2014 at 7:21 AM, Eli Bendersky <span dir="ltr"><<a href="mailto:eliben@google.com" target="_blank">eliben@google.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div><div class="h5">On Fri, Aug 1, 2014 at 6:26 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>>> With the approach of using the backend mangler, you wouldn't need to<br>
>> strip the \01 prefix because the backend mangler does that anyway (see<br>
>> lib/IR/Mangler.cpp:105).<br>
>><br>
><br>
> Yes, but this is not the path I'm going through - I don't have a LLVM<br>
> GlobalValue. The Twine-accepting function doesn't do this stripping. See<br>
> <a href="http://llvm.org/bugs/show_bug.cgi?id=20127" target="_blank">http://llvm.org/bugs/show_bug.cgi?id=20127</a> - I'd be able to simplify this<br>
> code when it's fixed.<br>
<br>
</div>Cool. Nice to have a testcase for the patch in pr20127 :-)<br>
<br>
Do you want me to apply it first?<br></blockquote><div><br></div></div></div><div>Yes, that would be great.</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Eli</div><div> </div></font></span></div></div>

</div>
</blockquote></div><br></div>