https://gcc.gnu.org/g:479a0a8644e5efa0470f032be21ca7749968af42
commit r15-8905-g479a0a8644e5efa0470f032be21ca7749968af42 Author: Jonathan Wakely <jwak...@redhat.com> Date: Tue Mar 25 00:27:52 2025 +0000 libstdc++: Allow std::ranges::to to create unions LWG 4229 points out that the std::ranges::to wording refers to class types, but I added an assertion using std::is_class_v which only allows non-union class types. LWG consensus is that unions should be allowed, so this additionally uses std::is_union_v. libstdc++-v3/ChangeLog: * include/std/ranges (ranges::to): Allow unions as well as non-union class types. * testsuite/std/ranges/conv/lwg4229.cc: New test. Reviewed-by: Tomasz KamiĆski <tkami...@redhat.com> Diff: --- libstdc++-v3/include/std/ranges | 4 ++-- libstdc++-v3/testsuite/std/ranges/conv/lwg4229.cc | 18 ++++++++++++++++++ 2 files changed, 20 insertions(+), 2 deletions(-) diff --git a/libstdc++-v3/include/std/ranges b/libstdc++-v3/include/std/ranges index 34c6f113e216..7a339c51368e 100644 --- a/libstdc++-v3/include/std/ranges +++ b/libstdc++-v3/include/std/ranges @@ -9421,7 +9421,7 @@ namespace __detail to [[nodiscard]] (_Rg&& __r, _Args&&... __args) { static_assert(!is_const_v<_Cont> && !is_volatile_v<_Cont>); - static_assert(is_class_v<_Cont>); + static_assert(is_class_v<_Cont> || is_union_v<_Cont>); if constexpr (__detail::__toable<_Cont, _Rg>) { @@ -9580,7 +9580,7 @@ namespace __detail to [[nodiscard]] (_Args&&... __args) { static_assert(!is_const_v<_Cont> && !is_volatile_v<_Cont>); - static_assert(is_class_v<_Cont>); + static_assert(is_class_v<_Cont> || is_union_v<_Cont>); using __detail::_To; using views::__adaptor::_Partial; diff --git a/libstdc++-v3/testsuite/std/ranges/conv/lwg4229.cc b/libstdc++-v3/testsuite/std/ranges/conv/lwg4229.cc new file mode 100644 index 000000000000..780ed1fd932b --- /dev/null +++ b/libstdc++-v3/testsuite/std/ranges/conv/lwg4229.cc @@ -0,0 +1,18 @@ +// { dg-do compile { target c++23 } } + +// LWG 4229 std::ranges::to with union return type + +#include <ranges> + +union U +{ + template<std::ranges::input_range R> U(std::from_range_t, R&&) { } + + int i; +}; + +void +test_lwg4229(std::ranges::subrange<int*> r) +{ + U u = std::ranges::to<U>(r); +}