[cfe-commits] [PR10113][patch] Don't consider template arguments when the visibility is already explicit

[Eli Friedman]

2012/1/1 Rafael Ávila de Espíndola <rafael.espindola at gmail.com>:
> The attached patch fixes PR10113 by disregarding template arguments in
> LV computation when we already have an explicit visibility.
>
> This is similar to what the code already does for types. Is it OK?

Disregarding the effect on visibility, this affects the linkage in
ways which are clearly wrong.

Anyway, I'm not actually convinced there's a bug here: unless I'm
mistaken, there isn't any legal way to refer to a type with hidden
visibility outside the shared library where it is defined.  Or am I
missing something?

-Eli