[PATCH] D34637: [libunwind] Add _LIBUNWIND_DISABLE_VISIBILITY_ANNOTATIONS

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 27 11:30:38 PDT 2017


compnerd accepted this revision.
compnerd added a comment.
This revision is now accepted and ready to land.

Discussed with @thakis on IRC; we can go ahead with this, but this will change in the future to be conditionalised on whether you are building a static or shared library.  The intention here is to build without the exports and maintain hidden visibility.  Although this is different from libc++, it is due to the fact that the current libunwind interfaces dont make as much sense on Windows (the itanium model and the windows exception model are thoroughly different), so removing the visibility attributes doesn't make much sense.


https://reviews.llvm.org/D34637





More information about the llvm-commits mailing list