On 26 Mar 2014 11:09, "Reid Kleckner" <[email protected]> wrote:
>
> LGTM
>
> On Wed, Mar 26, 2014 at 9:28 AM, Nico Rieck <[email protected]> wrote:
>>
>> Both attributes require external linkage. This is not fully compatible
>> with MSVC because MSVC allows exporting/importing from anonymous
>> namespaces. But until we can mangle this properly this should not be
>> allowed.
>
>
> 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
In C++11 onwards they have internal linkage, even.
> ....
> +
> +#if __has_feature(cxx_auto_type)
>
> Any reason for this #if? The test only uses c++11 and c++1y.
>
> +__declspec(dllexport) auto InternalAutoTypeGlobal = Internal(); //
expected-error{{'InternalAutoTypeGlobal' must have external linkage when
declared 'dllexport'}}
> +__declspec(dllexport) auto ExternalAutoTypeGlobal = External();
> +#endif
>
>
>
>
> _______________________________________________
> cfe-commits mailing list
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
