Skip to content

Commit 453620f

Browse files
committed
[libc++] Make _VSTD and alias for std
There is no practical difference between `_VSTD` and `std` so we should just remove `_VSTD`. This is the first step. Reviewed By: ldionne, #libc Spies: jeroen.dobbelaere, wmaxey, EricWF, lebedev.ri, __simt__, dim, mgrang, sstefan1, wenlei, smeenai, libcxx-commits, #libc_vendors Differential Revision: https://reviews.llvm.org/D117811
1 parent 676ddba commit 453620f

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

libcxx/docs/ReleaseNotes.rst

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,10 @@ API Changes
140140
instead. The ``<experimental/coroutine>`` header will be removed
141141
in LLVM 15.
142142

143+
- ``_VSTD`` is now an alias for ``std`` instead of ``std::_LIBCPP_ABI_NAMESPACE``.
144+
This is technically not a functional change, except for folks that might have been
145+
using ``_VSTD`` in creative ways (which has never been officially supported).
146+
143147
ABI Changes
144148
-----------
145149

libcxx/include/__config

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -821,10 +821,10 @@ typedef __char32_t char32_t;
821821
// Inline namespaces are available in Clang/GCC/MSVC regardless of C++ dialect.
822822
#define _LIBCPP_BEGIN_NAMESPACE_STD namespace std { inline namespace _LIBCPP_ABI_NAMESPACE {
823823
#define _LIBCPP_END_NAMESPACE_STD } }
824-
#define _VSTD std::_LIBCPP_ABI_NAMESPACE
824+
#define _VSTD std
825825
_LIBCPP_BEGIN_NAMESPACE_STD _LIBCPP_END_NAMESPACE_STD
826826

827-
#if _LIBCPP_STD_VER >= 17
827+
#if _LIBCPP_STD_VER > 14
828828
#define _LIBCPP_BEGIN_NAMESPACE_FILESYSTEM \
829829
_LIBCPP_BEGIN_NAMESPACE_STD inline namespace __fs { namespace filesystem {
830830
#else

0 commit comments

Comments
 (0)