[PATCH] D52662: [libc++] Make sure we can build libc++ with -fvisibility=hidden

[Eric Fiselier via Phabricator]

EricWF added inline comments.


================
Comment at: libcxx/include/locale:2426
+_LIBCPP_TIME_GET_STORAGE_EXPLICIT_INSTANTIATION(wchar_t)
+
 template <class _CharT, class _InputIterator = istreambuf_iterator<_CharT> >
----------------
Undef the macro after we're done using it.


================
Comment at: libcxx/include/thread:170
 
+extern template _LIBCPP_EXPORTED_FROM_ABI void __thread_specific_ptr<__thread_struct>::__at_thread_exit(void*);
+
----------------
So we likely only got this symbol in the dylib as a result of an implicit instantiation. Changing this to an explicit instantiation might be ABI breaking.


================
Comment at: libcxx/src/iostream.cpp:80
 
-ios_base::Init __start_std_streams;
+_LIBCPP_EXPORTED_FROM_ABI ios_base::Init __start_std_streams;
 
----------------
Does this actually need to be exported to serve it's function of initializing the streams?


Repository:
  rCXX libc++

https://reviews.llvm.org/D52662