[PATCH] D53868: Build with -fvisibility=hidden

Eric Fiselier via Phabricator reviews at reviews.llvm.org
Tue Oct 30 07:49:23 PDT 2018


EricWF created this revision.
EricWF added reviewers: ldionne, mclow.lists.
Herald added a subscriber: mgorny.

This change changes the build to use -fvisibility=hidden

The exports this patch removes are symbols that should have never been exported
by the dylib in the first place, and should all be symbols which the linker
won't de-duplicate across SO boundaries, making them safe to remove.

After this change, we should be able to apply `_LIBCPP_HIDDEN` to the versioning namespace without changing the export lists.


Repository:
  rCXX libc++

https://reviews.llvm.org/D53868

Files:
  CMakeLists.txt
  lib/abi/CHANGELOG.TXT
  lib/abi/x86_64-unknown-linux-gnu.v1.abilist

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D53868.171697.patch
Type: text/x-patch
Size: 390981 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20181030/63501af0/attachment-0001.bin>


More information about the libcxx-commits mailing list