[libcxx-commits] [PATCH] D153658: [libc++] Make everything in namespace std have default type visibility and hidden visibility and remove _LIBCPP_ENUM_VIS

Nikolas Klauser via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Jun 23 12:00:28 PDT 2023


philnik created this revision.
philnik added a reviewer: ldionne.
Herald added a project: All.
philnik requested review of this revision.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

This ensures that `-fvisibility` doesn't affect libc++ in any way other than the visibility for global functions (which should all be marked `_LIBCPP_HIDE_FROM_ABI`) and avoids having to add `_LIBCPP_ENUM_VIS`, since that is handled through `type_visibility` and GCC always makes the visibility of enums default.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D153658

Files:
  libcxx/.clang-format
  libcxx/docs/DesignDocs/VisibilityMacros.rst
  libcxx/include/__charconv/chars_format.h
  libcxx/include/__compare/ordering.h
  libcxx/include/__config
  libcxx/include/__filesystem/copy_options.h
  libcxx/include/__filesystem/directory_options.h
  libcxx/include/__filesystem/file_type.h
  libcxx/include/__filesystem/path.h
  libcxx/include/__filesystem/perm_options.h
  libcxx/include/__filesystem/perms.h
  libcxx/include/__format/format_arg.h
  libcxx/include/__format/formatter_output.h
  libcxx/include/__format/parser_std_format_spec.h
  libcxx/include/__fwd/subrange.h
  libcxx/include/new

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D153658.534044.patch
Type: text/x-patch
Size: 10666 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230623/929ac0ee/attachment.bin>


More information about the libcxx-commits mailing list