[PATCH] D55517: Remove `_VSTD`

JF Bastien via Phabricator reviews at reviews.llvm.org
Mon Dec 10 10:48:22 PST 2018


jfb added a comment.

In D55517#1325829 <https://reviews.llvm.org/D55517#1325829>, @ldionne wrote:

> In D55517#1325586 <https://reviews.llvm.org/D55517#1325586>, @jfb wrote:
>
> > > Quick Question: What's the difference between writing `_VSTD::` and `std::`? Nothing.
> >
> > I thought it was `std:: _LIBCPP_ABI_NAMESPACE` ?
>
>
> That's correct, however I don't think it's a big deal. Indeed, discussed it and we couldn't find a reason why we would ever want to call a function that is not in the current ABI's namespace. And even if we did, we could always use `std::<ABI namespace>::function`.


Sure, my point is that the commit message isn't right, and should be updated to have this rationale.


Repository:
  rCXX libc++

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55517/new/

https://reviews.llvm.org/D55517





More information about the libcxx-commits mailing list