<p dir="ltr"><br>
On 26 Mar 2014 11:09, "Reid Kleckner" <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
><br>
> LGTM<br>
><br>
> On Wed, Mar 26, 2014 at 9:28 AM, Nico Rieck <<a href="mailto:nico.rieck@gmail.com">nico.rieck@gmail.com</a>> wrote:<br>
>><br>
>> Both attributes require external linkage. This is not fully compatible<br>
>> with MSVC because MSVC allows exporting/importing from anonymous<br>
>> namespaces. But until we can mangle this properly this should not be<br>
>> allowed.<br>
><br>
>  <br>
> Yeah, I don't think we should support exporting/importing from anonymous namespaces.  They have external language linkage, but really they aren't externally visible and can't be imported/exported.  =P</p>

<p dir="ltr">In C++11 onwards they have internal linkage, even.</p>
<p dir="ltr">> ....<br>
> +<br>
> +#if __has_feature(cxx_auto_type)<br>
><br>
> Any reason for this #if?  The test only uses c++11 and c++1y.<br>
><br>
> +__declspec(dllexport) auto InternalAutoTypeGlobal = Internal(); // expected-error{{'InternalAutoTypeGlobal' must have external linkage when declared 'dllexport'}}<br>
> +__declspec(dllexport) auto ExternalAutoTypeGlobal = External();<br>
> +#endif<br>
><br>
><br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</p>