[libc++] Replace uses of _LIBCPP_ALWAYS_INLINE by _LIBCPP_INLINE_VISIBILITY
authorLouis Dionne <ldionne@apple.com>
Thu, 5 Jul 2018 16:49:38 +0000 (16:49 +0000)
committerLouis Dionne <ldionne@apple.com>
Thu, 5 Jul 2018 16:49:38 +0000 (16:49 +0000)
commit4a8f3f994821e752319b9c76ed772ec0c40b5d03
tree86d340f53f5390eb645f05444d55b02111914b3b
parent14c922a240a7dd9f4ade63f5dd319cbd19678dc6
[libc++] Replace uses of _LIBCPP_ALWAYS_INLINE by _LIBCPP_INLINE_VISIBILITY

Summary:
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.

Reviewers: EricWF

Subscribers: christof, llvm-commits, dexonsmith, erikvanderpoel, mclow.lists

Differential Revision: https://reviews.llvm.org/D48892

llvm-svn: 336369
30 files changed:
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