<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 15, 2016 at 9:44 AM, Mehdi Amini <span dir="ltr"><<a href="mailto:mehdi.amini@apple.com" target="_blank">mehdi.amini@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><br><div><span class="gmail-"><blockquote type="cite"><div>On Sep 15, 2016, at 9:35 AM, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>> wrote:</div><br class="gmail-m_-394393638303517582Apple-interchange-newline"><div><div dir="ltr"><div>BTW, that doesn't seem to be the only issue.<br><br></div><div>Right now patatino gets internalized, which fails the verifier, that would be fixed by  pr30396 or some workaround.<br><br></div><div>But, adding -upatatino to the command line causes gold to fail because foo is internalized. I can't quite make what is the code path that is missing, but nothing creates the llvm.compiler.used that should prevent foo from being removed.<br></div></div></div></blockquote><div><br></div></span><div>Indeed, it looks like the call to updateCompilerUsed got lost in the new API…</div><div><br></div><div>Teresa: for ThinLTO it got lost when you moved to the index-based internalization in <a href="http://reviews.llvm.org/D20290" target="_blank">http://reviews.llvm.org/<wbr>D20290</a> I believe.</div></div></div></blockquote><div><br></div><div>Do we need that for ThinLTO? Note the AsmUndefinedRefs are being preserved via the MustPreserveGV callback thinLTOInternalizeModule.</div><div><br></div><div>Teresa</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="word-wrap:break-word"><div><div><br></div><div><br></div><div>— </div><span class="gmail-HOEnZb"><font color="#888888"><div>Mehdi</div></font></span><span class="gmail-"><div><br></div><div><br></div><br><blockquote type="cite"><div><div dir="ltr"><div><br></div><div>Cheers,<br></div><div>Rafael<br><br></div><div><br><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 15 September 2016 at 11:27, 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><span></span><br><span></span><div class="gmail_extra"><div class="gmail_quote"><span><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote"><span>
</span>Yes that sounds the proper fix: we should have llvm-nm report the correct list for instance.<br></blockquote><div><br></div></span><div>Reported <a href="https://llvm.org/bugs/show_bug.cgi?id=30396" target="_blank">https://llvm.org/bugs/show_bug<wbr>.cgi?id=30396</a> to track it.<br><br></div><div>Cheers,<br></div><div>Rafael<br> <br></div></div></div></div>
</blockquote></div><br></div>
</div></blockquote></span></div><br></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><span style="font-family:times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top:2px solid rgb(213,15,37)">Teresa Johnson |</td><td nowrap style="border-top:2px solid rgb(51,105,232)"> Software Engineer |</td><td nowrap style="border-top:2px solid rgb(0,153,57)"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td nowrap style="border-top:2px solid rgb(238,178,17)"> 408-460-2413</td></tr></tbody></table></span></div>
</div></div>