[libcxx-commits] [libcxx] [libc++] Add alias template for __strip_signature::type (PR #181955)
Louis Dionne via libcxx-commits
libcxx-commits at lists.llvm.org
Mon Feb 23 08:06:34 PST 2026
https://github.com/ldionne approved this pull request.
> I'm not actively against this, but I'm also not convinced we need it, especially as a separate patch. I don't really want to set the precedent of accepting these kinds changes. If there were more than two places we used it in, maybe it'd improve readability enough, but as-is I'm not convinced.
I'm also neutral on this, but I think this patch makes the code more in line with how we generally try to write new code, so I think we should accept it. It might be helpful to document what our convention is around internal helpers and `_t` aliases in our CONTRIBUTING documentation.
https://github.com/llvm/llvm-project/pull/181955
More information about the libcxx-commits
mailing list