[Patch] (take 2) New EliminateAvailableExternally pass / Pass down -flto

Rafael EspĂ­ndola rafael.espindola at gmail.com
Sat Jul 11 13:07:51 PDT 2015


>
>
> Before my patch, code gen suppressed the emission of this function since
> it was available externally, and as a result, EmitVisibility, and
> thus MCSymbolELF::setVisibility, were simply never called. The undefined
> symbol then ended up with the default visibility. It seems to me that this
> essentially worked by luck.
>
>
Yes, that is a bug in codegen. If the function should not be hidden clang
should not be setting the visibility to hidden.

I will try to write a combined patch to change codegen and the new pass and
see what is wrong in clang.

Cheers,
Rafael
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150711/743a537e/attachment.html>


More information about the cfe-commits mailing list