[all-commits] [llvm/llvm-project] cb793e: [libc++][NFCI] Remove uses of _LIBCPP_INLINE_VAR

[Louis Dionne via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: cb793e1a3655b8a571cb621db9d221c46f32ba9f
      https://github.com/llvm/llvm-project/commit/cb793e1a3655b8a571cb621db9d221c46f32ba9f
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2021-09-22 (Wed, 22 Sep 2021)

  Changed paths:
    M libcxx/include/__config
    M libcxx/include/__functional/bind.h
    M libcxx/include/__memory/allocator_arg_t.h
    M libcxx/include/__memory/uses_allocator.h
    M libcxx/include/__mutex_base
    M libcxx/include/__utility/in_place.h
    M libcxx/include/__utility/piecewise_construct.h
    M libcxx/include/chrono
    M libcxx/include/experimental/simd
    M libcxx/include/new
    M libcxx/include/optional
    M libcxx/include/ratio
    M libcxx/include/system_error
    M libcxx/include/tuple
    M libcxx/include/type_traits
    M libcxx/include/variant

  Log Message:
  -----------
  [libc++][NFCI] Remove uses of _LIBCPP_INLINE_VAR

All supported compilers provide support for inline variables in C++17 now.
Also, as a fly-by fix, replace some uses of _LIBCPP_CONSTEXPR by just
constexpr.

The only exception in this patch is `std::ignore`, which is provided
prior to C++17. Since it is defined in an anonymous namespace, it always
has internal linkage anyway, so using an inline variable there doesn't
provide any benefit. Instead, `inline` was removed entirely on `std::ignore`.

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