[PATCH] D24679: [libc++] Fix extern template visibility for Windows

Shoaib Meenai via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 19 11:20:35 PDT 2016

smeenai added inline comments.

Comment at: include/__config:559
@@ -554,1 +558,3 @@
compnerd wrote:
> Does it make sense for `_LIBCPP_CLASS_TEMPLATE_INSTANTIATION_VIS` to ever be marked as `__declspec(dllimport)`?  The macro is applied on instantiations in the implementation, not the header, so this shouldn't be visible to users.
It doesn't, which is why I always made it expand to either `__declspec(dllexport)` or empty.


More information about the cfe-commits mailing list