* [Bug c++/100639] reverse_view<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I>
2021-05-17 16:50 [Bug c++/100639] New: reverse_view<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I> barry.revzin at gmail dot com
@ 2021-05-17 18:03 ` redi at gcc dot gnu.org
2021-05-18 18:22 ` [Bug libstdc++/100639] reverse_iterator<I>::reference " ppalka at gcc dot gnu.org
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: redi at gcc dot gnu.org @ 2021-05-17 18:03 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100639
Jonathan Wakely <redi at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Last reconfirmed| |2021-05-17
Status|UNCONFIRMED |NEW
Ever confirmed|0 |1
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug libstdc++/100639] reverse_iterator<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I>
2021-05-17 16:50 [Bug c++/100639] New: reverse_view<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I> barry.revzin at gmail dot com
2021-05-17 18:03 ` [Bug c++/100639] " redi at gcc dot gnu.org
@ 2021-05-18 18:22 ` ppalka at gcc dot gnu.org
2021-05-18 19:42 ` ppalka at gcc dot gnu.org
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: ppalka at gcc dot gnu.org @ 2021-05-18 18:22 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100639
Patrick Palka <ppalka at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |ppalka at gcc dot gnu.org
Status|NEW |ASSIGNED
Assignee|unassigned at gcc dot gnu.org |ppalka at gcc dot gnu.org
Summary|reverse_view<I>::reference |reverse_iterator<I>::refere
|erroneously uses |nce erroneously uses
|iterator_traits<I>::referen |iterator_traits<I>::referen
|ce instead of |ce instead of
|iter_reference_t<I> |iter_reference_t<I>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug libstdc++/100639] reverse_iterator<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I>
2021-05-17 16:50 [Bug c++/100639] New: reverse_view<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I> barry.revzin at gmail dot com
2021-05-17 18:03 ` [Bug c++/100639] " redi at gcc dot gnu.org
2021-05-18 18:22 ` [Bug libstdc++/100639] reverse_iterator<I>::reference " ppalka at gcc dot gnu.org
@ 2021-05-18 19:42 ` ppalka at gcc dot gnu.org
2021-05-20 18:11 ` cvs-commit at gcc dot gnu.org
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: ppalka at gcc dot gnu.org @ 2021-05-18 19:42 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100639
--- Comment #1 from Patrick Palka <ppalka at gcc dot gnu.org> ---
The assertion also fails when __int128 isn't available (which can be simulated
by compiling with -U__SIZEOF_INT128__), because in that case iota_view's
difference_type is the integer-like class type __max_diff_type, and
integer-like class types aren't integral even in GNU mode.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug libstdc++/100639] reverse_iterator<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I>
2021-05-17 16:50 [Bug c++/100639] New: reverse_view<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I> barry.revzin at gmail dot com
` (2 preceding siblings ...)
2021-05-18 19:42 ` ppalka at gcc dot gnu.org
@ 2021-05-20 18:11 ` cvs-commit at gcc dot gnu.org
2021-06-10 18:43 ` cvs-commit at gcc dot gnu.org
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2021-05-20 18:11 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100639
--- Comment #2 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Patrick Palka <ppalka@gcc.gnu.org>:
https://gcc.gnu.org/g:d5cbe0f0d4b7bc11f80b2236521f90ec94e95767
commit r12-946-gd5cbe0f0d4b7bc11f80b2236521f90ec94e95767
Author: Patrick Palka <ppalka@redhat.com>
Date: Thu May 20 14:08:17 2021 -0400
libstdc++: Implement missing P0896R4 changes to reverse_iterator [PR100639]
This implements the P0896R4 changes to reverse_iterator's member types
value_type, difference_type and reference in C++20 mode, which fixes
taking the reverse_iterator of an iterator with a non-integral
difference_type (such as iota_view<long long>).
libstdc++-v3/ChangeLog:
PR libstdc++/100639
* include/bits/stl_iterator.h (reverse_iterator::difference_type):
In C++20 mode, define in terms of iter_difference_t as per P0896R4.
(reverse_iterator::reference): Likewise, but with iter_reference_t.
(reverse_iterator::value_type): Likewise, but with iter_value_t.
* testsuite/std/ranges/adaptors/reverse.cc (test08): New test.
* testsuite/24_iterators/reverse_iterator/100639.cc: New test.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug libstdc++/100639] reverse_iterator<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I>
2021-05-17 16:50 [Bug c++/100639] New: reverse_view<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I> barry.revzin at gmail dot com
` (3 preceding siblings ...)
2021-05-20 18:11 ` cvs-commit at gcc dot gnu.org
@ 2021-06-10 18:43 ` cvs-commit at gcc dot gnu.org
2021-06-10 18:49 ` cvs-commit at gcc dot gnu.org
2021-06-10 18:50 ` ppalka at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2021-06-10 18:43 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100639
--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-11 branch has been updated by Patrick Palka
<ppalka@gcc.gnu.org>:
https://gcc.gnu.org/g:a50cc70f1b5ce3ae11528095fa3a1feadb206357
commit r11-8540-ga50cc70f1b5ce3ae11528095fa3a1feadb206357
Author: Patrick Palka <ppalka@redhat.com>
Date: Thu May 20 14:08:17 2021 -0400
libstdc++: Implement missing P0896R4 changes to reverse_iterator [PR100639]
This implements the P0896R4 changes to reverse_iterator's member types
value_type, difference_type and reference in C++20 mode, which fixes
taking the reverse_iterator of an iterator with a non-integral
difference_type (such as iota_view<long long>).
libstdc++-v3/ChangeLog:
PR libstdc++/100639
* include/bits/stl_iterator.h (reverse_iterator::difference_type):
In C++20 mode, define in terms of iter_difference_t as per P0896R4.
(reverse_iterator::reference): Likewise, but with iter_reference_t.
(reverse_iterator::value_type): Likewise, but with iter_value_t.
* testsuite/std/ranges/adaptors/reverse.cc (test08): New test.
* testsuite/24_iterators/reverse_iterator/100639.cc: New test.
(cherry picked from commit d5cbe0f0d4b7bc11f80b2236521f90ec94e95767)
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug libstdc++/100639] reverse_iterator<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I>
2021-05-17 16:50 [Bug c++/100639] New: reverse_view<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I> barry.revzin at gmail dot com
` (4 preceding siblings ...)
2021-06-10 18:43 ` cvs-commit at gcc dot gnu.org
@ 2021-06-10 18:49 ` cvs-commit at gcc dot gnu.org
2021-06-10 18:50 ` ppalka at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2021-06-10 18:49 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100639
--- Comment #4 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Patrick Palka
<ppalka@gcc.gnu.org>:
https://gcc.gnu.org/g:c8bd39bc10eac3028dbf509b27457172360e6e8c
commit r10-9902-gc8bd39bc10eac3028dbf509b27457172360e6e8c
Author: Patrick Palka <ppalka@redhat.com>
Date: Thu May 20 14:08:17 2021 -0400
libstdc++: Implement missing P0896R4 changes to reverse_iterator [PR100639]
This implements the P0896R4 changes to reverse_iterator's member types
value_type, difference_type and reference in C++20 mode, which fixes
taking the reverse_iterator of an iterator with a non-integral
difference_type (such as iota_view<long long>).
libstdc++-v3/ChangeLog:
PR libstdc++/100639
* include/bits/stl_iterator.h (reverse_iterator::difference_type):
In C++20 mode, define in terms of iter_difference_t as per P0896R4.
(reverse_iterator::reference): Likewise, but with iter_reference_t.
(reverse_iterator::value_type): Likewise, but with iter_value_t.
* testsuite/std/ranges/adaptors/reverse.cc (test08): New test.
* testsuite/24_iterators/reverse_iterator/100639.cc: New test.
(cherry picked from commit d5cbe0f0d4b7bc11f80b2236521f90ec94e95767)
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug libstdc++/100639] reverse_iterator<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I>
2021-05-17 16:50 [Bug c++/100639] New: reverse_view<I>::reference erroneously uses iterator_traits<I>::reference instead of iter_reference_t<I> barry.revzin at gmail dot com
` (5 preceding siblings ...)
2021-06-10 18:49 ` cvs-commit at gcc dot gnu.org
@ 2021-06-10 18:50 ` ppalka at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: ppalka at gcc dot gnu.org @ 2021-06-10 18:50 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=100639
Patrick Palka <ppalka at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |10.4
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
--- Comment #5 from Patrick Palka <ppalka at gcc dot gnu.org> ---
Fixed for 10.4/11.2/12
^ permalink raw reply [flat|nested] 8+ messages in thread