Re: [PATCH] D24065: [libc++] Don't attempt to dllexport enum class

2016-09-01 Thread Saleem Abdulrasool via cfe-commits
compnerd added a comment. Ugh, I was wrong. There is type information associated with a scoped enum. This means that `_LIBCPP_TYPE_VIS` is the right annotation here. https://reviews.llvm.org/D24065 ___ cfe-commits mailing list cfe-commits@lists.l

Re: [PATCH] D24065: [libc++] Don't attempt to dllexport enum class

2016-08-30 Thread Shoaib Meenai via cfe-commits
smeenai marked 3 inline comments as done. smeenai added a comment. https://reviews.llvm.org/D24065 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D24065: [libc++] Don't attempt to dllexport enum class

2016-08-30 Thread Shoaib Meenai via cfe-commits
smeenai updated this revision to Diff 69797. smeenai updated the summary for this revision. smeenai added a comment. Removing export entirely, per compnerd's suggestion https://reviews.llvm.org/D24065 Files: include/__config Index: include/__config ===

Re: [PATCH] D24065: [libc++] Don't attempt to dllexport enum class

2016-08-30 Thread Saleem Abdulrasool via cfe-commits
compnerd added inline comments. Comment at: include/__config:719 @@ -718,3 +718,3 @@ #else // _LIBCPP_HAS_NO_STRONG_ENUMS -#define _LIBCPP_DECLARE_STRONG_ENUM(x) enum class _LIBCPP_TYPE_VIS x +#define _LIBCPP_DECLARE_STRONG_ENUM(x) enum class _LIBCPP_TYPE_VIS_ONLY x #define _LI

Re: [PATCH] D24065: [libc++] Don't attempt to dllexport enum class

2016-08-30 Thread Shoaib Meenai via cfe-commits
smeenai added inline comments. Comment at: include/__config:719 @@ -718,3 +718,3 @@ #else // _LIBCPP_HAS_NO_STRONG_ENUMS -#define _LIBCPP_DECLARE_STRONG_ENUM(x) enum class _LIBCPP_TYPE_VIS x +#define _LIBCPP_DECLARE_STRONG_ENUM(x) enum class _LIBCPP_TYPE_VIS_ONLY x #define _LIB

Re: [PATCH] D24065: [libc++] Don't attempt to dllexport enum class

2016-08-30 Thread Saleem Abdulrasool via cfe-commits
compnerd added inline comments. Comment at: include/__config:719 @@ -718,3 +718,3 @@ #else // _LIBCPP_HAS_NO_STRONG_ENUMS -#define _LIBCPP_DECLARE_STRONG_ENUM(x) enum class _LIBCPP_TYPE_VIS x +#define _LIBCPP_DECLARE_STRONG_ENUM(x) enum class _LIBCPP_TYPE_VIS_ONLY x #define _LI