[libcxx-commits] [libcxx] [libcxx] Replace a few _LIBCPP_INLINE_VISIBILITY with _LIBCPP_HIDE_FR… (PR #66661)

via libcxx-commits libcxx-commits at lists.llvm.org
Mon Sep 18 09:02:21 PDT 2023


philnik777 wrote:

The key word here is "gradually". As you noticed yourself, this patch is really big, which means quite likely a bunch of merge conflicts for relatively little benefit. From a quick grep, it looks like this diff affects over 5000 instances. I agree it would be nice to remove all the `_LIBCPP_INLINE_VISIBILITY`, but I'm not convinced the improvement here is worth the cost.

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


More information about the libcxx-commits mailing list