[all-commits] [llvm/llvm-project] 16d105: [libc++] Remove _LIBCPP_METHOD_TEMPLATE_IMPLICIT_I...

[Nikolas Klauser via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 16d10546d29355f796cbdb307f0f89d6679d14a0
      https://github.com/llvm/llvm-project/commit/16d10546d29355f796cbdb307f0f89d6679d14a0
  Author: Nikolas Klauser <nikolasklauser at berlin.de>
  Date:   2025-04-08 (Tue, 08 Apr 2025)

  Changed paths:
    M libcxx/.clang-format
    M libcxx/docs/DesignDocs/VisibilityMacros.rst
    M libcxx/include/__condition_variable/condition_variable.h
    M libcxx/include/__config
    M libcxx/include/__locale
    M libcxx/include/__thread/thread.h
    M libcxx/include/condition_variable
    M libcxx/include/future
    M libcxx/include/locale
    M libcxx/include/mutex
    M libcxx/include/shared_mutex
    M libcxx/include/string

  Log Message:
  -----------
  [libc++] Remove _LIBCPP_METHOD_TEMPLATE_IMPLICIT_INSTANTIATION_VIS (#111964)

This macro isn't required if we define all the functions inline. In
fact, quite a few of the marked functions have already been inlined.

This patch basically only moves code around and adds
`_LIBCPP_HIDE_FROM_ABI` to the places where it's been missing so far.

This also removes inlining hints, since it dropps `inline` in some
places, but that shouldn't make much of a difference. The functions tend
to be either really small, so should be inlined anyways, or are big
enough that they shouldn't be inlined even with an inlinehint.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications