[PATCH] D48892: [libc++] Replace uses of _LIBCPP_ALWAYS_INLINE by _LIBCPP_INLINE_VISIBILITY

[Louis Dionne via Phabricator via llvm-commits]

ldionne created this revision.
ldionne added a reviewer: EricWF.
Herald added subscribers: llvm-commits, christof.

We never actually mean to always inline a function -- all the uses of
the macro I could find are actually attempts to control the visibility
of symbols. This is better described by _LIBCPP_INLINE_VISIBILITY, which
is actually always defined the same.

This change is orthogonal to the decision of what we're actually going
to do with _LIBCPP_INLINE_VISIBILITY -- it just simplifies things by
having one canonical way of doing things.


Repository:
  rL LLVM

https://reviews.llvm.org/D48892

Files:
  libcxx/docs/DesignDocs/VisibilityMacros.rst
  libcxx/include/__bsd_locale_fallbacks.h
  libcxx/include/__config
  libcxx/include/__locale
  libcxx/include/__nullptr
  libcxx/include/any
  libcxx/include/cmath
  libcxx/include/codecvt
  libcxx/include/exception
  libcxx/include/experimental/dynarray
  libcxx/include/experimental/filesystem
  libcxx/include/functional
  libcxx/include/future
  libcxx/include/initializer_list
  libcxx/include/ios
  libcxx/include/locale
  libcxx/include/math.h
  libcxx/include/memory
  libcxx/include/new
  libcxx/include/ostream
  libcxx/include/regex
  libcxx/include/stdexcept
  libcxx/include/streambuf
  libcxx/include/support/android/locale_bionic.h
  libcxx/include/support/xlocale/__posix_l_fallback.h
  libcxx/include/support/xlocale/__strtonum_fallback.h
  libcxx/include/system_error
  libcxx/include/typeinfo
  libcxx/include/vector
  libcxx/src/support/win32/thread_win32.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48892.153958.patch
Type: text/x-patch
Size: 87692 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180703/1fc06bb8/attachment.bin>