[libcxx-commits] [libcxx] [libc++] Remove workaround which allows setting _LIBCPP_OVERRIDABLE_FUNC_VIS externally (PR #113139)

[Nikolas Klauser via libcxx-commits]

philnik777 wrote:

> CC @llvm/libcxx-vendors
> 
> @philnik777 Can you please add a short explanation of the history of this flag? I think you removed it and that broke a use case, and now it can be removed. What broke when you originally removed it?

Some people used a custom new/delete, which requires them to have default visibility, but didn't want to leak any symbols otherwise. Since the new/delete visibility can now be controlled by a compiler flag, there is no reason to set it with out macro.

https://github.com/llvm/llvm-project/pull/113139