diff options
author | Patrick Palka <ppalka@redhat.com> | 2025-09-04 09:57:36 -0400 |
---|---|---|
committer | Patrick Palka <ppalka@redhat.com> | 2025-09-04 09:57:36 -0400 |
commit | 7f7f1878eedd8093d382e1e7b74649d7e97d5918 (patch) | |
tree | 3ea4d53be21a3723c41c4b67cdf22434b9ca8a92 /libgo | |
parent | 1a41e52d7ecb583947a37b8d2e16a0a9ea3777a4 (diff) | |
download | gcc-7f7f1878eedd8093d382e1e7b74649d7e97d5918.zip gcc-7f7f1878eedd8093d382e1e7b74649d7e97d5918.tar.gz gcc-7f7f1878eedd8093d382e1e7b74649d7e97d5918.tar.bz2 |
libstdc++: Conditionalize LWG 3569 changes to join_view
LWG 3569 adjusted join_view's iterator specification to handle non
default-constructible iterators by wrapping the corresponding data member
in std::optional, which we followed suit in r13-2649-g7aa80c82ecf3a3.
But this wrapping is unnecessary for iterators that are already
default-constructible. Rather than unconditionally using std::optional
here, which introduces time/space overhead, this patch conditionalizes
our LWG 3569 changes on the iterator in question being non-forward (and
thus non default-constructible). We check forwardness instead of
default-constructibility in order to accommodate input-only iterators
that satisfy but do not model default_initializable, e.g. whose default
constructor is underconstrained.
libstdc++-v3/ChangeLog:
* include/std/ranges (join_view::_Iterator::_M_satisfy):
Adjust to handle non-std::optional _M_inner as per before LWG 3569.
(join_view::_Iterator::_M_get_inner): New.
(join_view::_Iterator::_M_inner): Don't wrap in std::optional if
the iterator is forward. Initialize.
(join_view::_Iterator::operator*): Use _M_get_inner instead
of *_M_inner.
(join_view::_Iterator::operator++): Likewise.
(join_view::_Iterator::iter_move): Likewise.
(join_view::_Iterator::iter_swap): Likewise.
Reviewed-by: Tomasz KamiĆski <tkaminsk@redhat.com>
Reviewed-by: Jonathan Wakely <jwakely@redhat.com>
Diffstat (limited to 'libgo')
0 files changed, 0 insertions, 0 deletions