Revert "[libc++] Remove workaround which allows setting _LIBCPP_OVERRIDABLE_FUNC_VIS externally (#113139)"

[nico]

This reverts commit 2e686d6.

See #117571

[llvmbot]

@llvm/pr-subscribers-libcxx

Author: Nico Weber (nico)

Changes

…IDABLE_FUNC_VIS externally (#113139)"

This reverts commit 2e686d6.

See #117571

---

Full diff: https://github.com/llvm/llvm-project/pull/117779.diff

1 Files Affected:

• (modified) libcxx/include/__config (+5-1)

diff --git a/libcxx/include/__config b/libcxx/include/__config
index 9db00cd0c9fb93..fe01b58b8e6274 100644
--- a/libcxx/include/__config
+++ b/libcxx/include/__config
@@ -402,7 +402,11 @@ typedef __char32_t char32_t;
 #    define _LIBCPP_EXPORTED_FROM_ABI _LIBCPP_VISIBILITY("default")
 #    define _LIBCPP_EXTERN_TEMPLATE_TYPE_VIS _LIBCPP_VISIBILITY("default")
 #    define _LIBCPP_CLASS_TEMPLATE_INSTANTIATION_VIS
-#    define _LIBCPP_OVERRIDABLE_FUNC_VIS _LIBCPP_VISIBILITY("default")
+
+// TODO: Make this a proper customization point or remove the option to override it.
+#    ifndef _LIBCPP_OVERRIDABLE_FUNC_VIS
+#      define _LIBCPP_OVERRIDABLE_FUNC_VIS _LIBCPP_VISIBILITY("default")
+#    endif
 
 #    if !defined(_LIBCPP_DISABLE_VISIBILITY_ANNOTATIONS)
 // The inline should be removed once PR32114 is resolved

[ldionne]

LGTM but I'd like @philnik777 to stamp this as well. To unblock you, feel free to merge if @philnik777 hasn't seen this within the next 24h.

[nico]

Thanks!