[PATCH] Ignore/drop visibility for symbols with local linkage

Rafael EspĂ­ndola rafael.espindola at gmail.com
Wed May 7 10:35:21 PDT 2014

> Weird.  Reattached.  Try again?
> Just looking at this again, should the assert be stronger?  I.e., is
> this even better?
>     assert(!GlobalValue::isLocalLinkage(LT) &&
>            "Symbols with local linkage should not be merged");

The stronger version of the assert LGTM.


More information about the llvm-commits mailing list