[libcxx-commits] [PATCH] D117811: [libc++] Remove _VSTD
Louis Dionne via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Fri Jan 21 10:46:13 PST 2022
ldionne added a comment.
In D117811#3261812 <https://reviews.llvm.org/D117811#3261812>, @__simt__ wrote:
> I've mentioned this before, but NVIDIA defines _VSTD to something else in our product branch. This diff, here, implies that we will need a matching & opposite diff to further separate us from upstream.
>
> I'm not necessarily saying you have to do something convenient for NVIDIA, but know how it's not ideal.
I would like to understand what your use case is so we can either support it properly, or not at all. The current situation where we're cargo culting the usefulness of `_VSTD` based on some vendors saying "don't make this change, it'll break us" is not good. I would like a concrete and actionable description of how you are using it so we can either decide "okay, that is a supported use case of libc++ and we're going to commit to supporting that properly", or to the contrary "this is twisted and we don't commit to supporting that".
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D117811/new/
https://reviews.llvm.org/D117811
More information about the libcxx-commits
mailing list