<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 11 September 2016 at 21:14, Sam Shepperd via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</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">phabricss added a subscriber: phabricss.<br>
phabricss added a comment.<br>
<br>
It is still impossible to compile glibc with clang due to this bug.  Could this patch be reviewed please.<br></blockquote><div><br></div><div>Firstly, I thought glibc had applied a patch to fix this bug? As in, the error is correct and glibc fixed their bug?</div><div><br></div><div>As for the patch, its goal is to only demote an error to warning when the function with conflicting asm labels is never used. That's pretty clearly a workaround (if you ever call <span style="font-size:12.8px">acoshl, then you'll get an error</span>), but moreover the check is implemented incorrectly. "New->isUsed()" will be performed at the moment the new declaration is being parsed; of course it hasn't been used yet. isUsed() will turn true only after it's used, after we actually parse the rest of the code and build AST for the uses of the new declaration. As I suggested in my last review comment, you would need to defer the check until ActOnEndOfTranslationUnit in order to correctly answer New->isUsed().</div><div><br></div><div>Or just not try to implement this workaround, and instead change how codegen works so that we emit calls to functions with the same asm label that gcc would choose.</div><div><br></div></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<a href="https://reviews.llvm.org/D16171" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D16171</a><br>
<div class="gmail-HOEnZb"><div class="gmail-h5"><br>
<br>
<br>
______________________________<wbr>_________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/cfe-commits</a><br>
</div></div></blockquote></div><br></div></div>