https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
--- Comment #13 from Mikael Persson ---
Ah right lifetime
On Sun, 19 Dec 2021, 16:02 Mikael Persson,
wrote:
> Looking at it now, not sure why i didnt.
>
> On Sun, 19 Dec 2021, 14:00 redi at gcc dot gnu.org, <
> gcc-bugzi...@gcc.gnu.org>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
--- Comment #12 from Mikael Persson ---
Looking at it now, not sure why i didnt.
On Sun, 19 Dec 2021, 14:00 redi at gcc dot gnu.org, <
gcc-bugzi...@gcc.gnu.org> wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
>
> --- Comment #11
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
--- Comment #11 from Jonathan Wakely ---
Can't you just change your constructor to this?
zip(Containers&&... containers) :
containers_(std::forward(containers)...) { }
The problematic deduction seems to only happen because of the odd
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
--- Comment #10 from Mikael Persson ---
Thats great.
Saw the dates on the other pr, jikes this has been around a while.
Testing
z.containers_[0]->i is clever, ill add a test for that ( or more generally
z.containers_[0]->begin(), static_assert
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
--- Comment #9 from Jonathan Wakely ---
GCC 10.1 isn't using the deduction guide at all, it seems to prefer to deduce
the class type using the zip(Containers...) constructor, which deduces
non-reference types.
If I change the deduction guide
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
Jonathan Wakely changed:
What|Removed |Added
Keywords|needs-reduction |
--- Comment #8 from Jonathan Wakely
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
--- Comment #7 from Jonathan Wakely ---
Before that commit, the zip(as, bs) deduced the type as zip,
vector>, rather than zip&, vector&>.
So:
(In reply to Mikael Persson from comment #0)
> GCC does not change values in tuple of references
Is
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
--- Comment #6 from Andrew Pinski ---
(In reply to Andrew Pinski from comment #5)
> (In reply to Jonathan Wakely from comment #4)
> > Fixed by r11-1571 and r10-8343, although I'm not sure how that affected it.
>
> I suspect it is the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
--- Comment #5 from Andrew Pinski ---
(In reply to Jonathan Wakely from comment #4)
> Fixed by r11-1571 and r10-8343, although I'm not sure how that affected it.
I suspect it is the tsubst_pack_expansion change.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
Jonathan Wakely changed:
What|Removed |Added
Keywords|needs-bisection |
--- Comment #4 from Jonathan Wakely
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
Andrew Pinski changed:
What|Removed |Added
Keywords||needs-bisection,
|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
Martin Liška changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91780
--- Comment #1 from Mikael Persson ---
To clarify it also does not work in gcc trunk.
13 matches
Mail list logo