Looks great, thanks. On Thu, May 30, 2013 at 2:08 PM, Rafael Espíndola < [email protected]> wrote:
> Hi Richard, > > Attached is a new version with the suggestions you had on irc. > > On 29 May 2013 22:19, Rafael Espíndola <[email protected]> wrote: > > The testcase in PR16060 points out that while template arguments can > > show that a type is not externally visible, the standards still says > > they have external linkage. > > > > In terms of our implementation, it means that we should merge just the > > isExternallyVisible bit, not the formal linkage. > > > > This patch does just that. Suggestion for a less confusing name than > > LinkageVisibility are welcome! > > > > Cheers, > > Rafael >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
