[libc++abi] Do not export some implementation-detail functions
authorLouis Dionne <ldionne@apple.com>
Thu, 3 Oct 2019 14:24:53 +0000 (14:24 +0000)
committerLouis Dionne <ldionne@apple.com>
Thu, 3 Oct 2019 14:24:53 +0000 (14:24 +0000)
commit6f9459f7fe7db16b10ad1700fbb5c833e3dd4fec
tree22ebebbefbdf934f1976920906df694fd896d311
parentf6c34de11767fce0c6503925bcef7228f36c6842
[libc++abi] Do not export some implementation-detail functions

Summary:
Those functions started being mistakenly exported from the libc++abi
shared library after commit r344152 in 2018. Removing these symbols is
technically an ABI break. However, they are not part of the C++ ABI,
they haven't ever been re-exported from libc++, and they are not
declared in any public header, so it's very unlikely that calls to
these functions exist out there. Also, the functions have reserved
names, so any impacted user would have to have tried really hard
being broken by this removal.

Note that avoiding this kind of problem is exactly why we're now
controlling exported symbols explicitly with a textual list.

Also note that applying the hidden visibility attribute is necessary
because the list of exported symbols is only used on Apple platforms
for the time being.

Reviewers: phosek, mclow.lists, EricWF

Subscribers: christof, jkorous, dexonsmith, libcxx-commits

Tags: #libc

Differential Revision: https://reviews.llvm.org/D68357

llvm-svn: 373602
libcxxabi/lib/itanium-base.exp
libcxxabi/src/cxa_exception.h