Looks great, thanks.<br><br><div class="gmail_quote">On Thu, May 30, 2013 at 2:08 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Richard,<br>
<br>
Attached is a new version with the suggestions you had on irc.<br>
<div class="HOEnZb"><div class="h5"><br>
On 29 May 2013 22:19, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> The testcase in PR16060 points out that while template arguments can<br>
> show that a type is not externally visible, the standards still says<br>
> they have external linkage.<br>
><br>
> In terms of our implementation, it means that we should merge just the<br>
> isExternallyVisible bit, not the formal linkage.<br>
><br>
> This patch does just that. Suggestion for a less confusing name than<br>
> LinkageVisibility are welcome!<br>
><br>
> Cheers,<br>
> Rafael<br>
</div></div></blockquote></div><br>