[libc++abi] Do not export some implementation-detail functions

[ldionne]

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.

[smeenai]

Do we not build libc++abi with -fvisibility=hidden? That would also have prevented this, right?

[phosek]

LGTM

[ldionne]

In D68357#1692080, @smeenai wrote:

Do we not build libc++abi with -fvisibility=hidden? That would also have prevented this, right?

No, we don’t. At least we used not to. And yes, it would have prevented that.

libcxxabi/trunk/lib/itanium-base.exp

 __ZSt10unexpectedv
 __ZSt13get_terminatev
 __ZSt13set_terminatePFvvE
 __ZSt14get_unexpectedv
 __ZSt14set_unexpectedPFvvE
 __ZSt15get_new_handlerv
 __ZSt15set_new_handlerPFvvE
 __ZSt9terminatev
-
-# Misc
-__ZN10__cxxabiv119__getExceptionClassEPK17_Unwind_Exception
-__ZN10__cxxabiv119__setExceptionClassEP17_Unwind_Exceptiony
-__ZN10__cxxabiv121__isOurExceptionClassEPK17_Unwind_Exception

libcxxabi/trunk/src/cxa_exception.h

 #include "unwind.h"
 
 namespace __cxxabiv1 {
 
 static const uint64_t kOurExceptionClass          = 0x434C4E47432B2B00; // CLNGC++\0
 static const uint64_t kOurDependentExceptionClass = 0x434C4E47432B2B01; // CLNGC++\1
 static const uint64_t get_vendor_and_language     = 0xFFFFFFFFFFFFFF00; // mask for CLNGC++
 
-uint64_t __getExceptionClass  (const _Unwind_Exception*);
-void     __setExceptionClass  (      _Unwind_Exception*, uint64_t);
-bool     __isOurExceptionClass(const _Unwind_Exception*);
+_LIBCXXABI_HIDDEN uint64_t __getExceptionClass  (const _Unwind_Exception*);
+_LIBCXXABI_HIDDEN void     __setExceptionClass  (      _Unwind_Exception*, uint64_t);
+_LIBCXXABI_HIDDEN bool     __isOurExceptionClass(const _Unwind_Exception*);
 
 struct _LIBCXXABI_HIDDEN __cxa_exception {
 #if defined(__LP64__) || defined(_LIBCXXABI_ARM_EHABI)
     // This is a new field to support C++ 0x exception_ptr.
     // For binary compatibility it is at the start of this
     // struct which is prepended to the object thrown in
     // __cxa_allocate_exception.
     size_t referenceCount;