<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
gcc and MSVC both intentionally* retain the language linkage, and
they accept both test cases in the PR.<br>
<br>
I already explained in PR17337 why this special handling is
important, and it becomes more so in the face of nested language
linkage specifications:<br>
<br>
<a class="moz-txt-link-freetext" href="http://llvm.org/bugs/show_bug.cgi?id=17337">http://llvm.org/bugs/show_bug.cgi?id=17337</a><br>
<br>
The previous release of clang correctly accepted the code, just like
every other C++ compiler, up until checking was added in r171139 /
r181163 so this is a recent rejects-valid regression.<br>
<br>
clang r171139 states "The language linkage of redeclarations must
match. GCC was already reporting an error for this."<br>
<br>
If you don't feel comfortable reviewing this patch could you give
others a chance to take a look instead of marking the PR Invalid?<br>
<br>
* <a class="moz-txt-link-freetext" href="http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020">http://gcc.gnu.org/bugzilla/show_bug.cgi?id=41020</a><br>
<br>
<br>
<br>
<br>
<div class="moz-cite-prefix">On 24/09/2013 14:43, Richard Smith
wrote:<br>
</div>
<blockquote
cite="mid:CAOfiQqmoUbG6mwpfcAZvvrOw2=gPssWcu+ukf0FRzc2a2-moTw@mail.gmail.com"
type="cite">
<div dir="ltr">On Tue, Sep 24, 2013 at 3:00 AM, Alp Toker <span
dir="ltr"><<a moz-do-not-send="true"
href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span>
wrote:<br>
<div class="gmail_extra">
<div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
The previous language linkage needs to be retained as
described in<br>
[class.friend]p3 otherwise valid friend redeclarations
differing only in<br>
lexical linkage are rejected.<br>
</blockquote>
<div><br>
</div>
<div>[class.friend]p3 says the previous linkage
(internal/external/no linkage) is retained, not the
previous language linkage. Our current behavior appears to
be correct here.</div>
<div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
Fixes PR17337.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
<a moz-do-not-send="true" href="http://www.nuanti.com"
target="_blank">http://www.nuanti.com</a><br>
the browser experts<br>
<br>
</font></span><br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a moz-do-not-send="true"
href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a moz-do-not-send="true"
href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits"
target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br>
</blockquote>
</div>
<br>
</div>
</div>
</blockquote>
<br>
<pre class="moz-signature" cols="72">--
<a class="moz-txt-link-freetext" href="http://www.nuanti.com">http://www.nuanti.com</a>
the browser experts
</pre>
</body>
</html>