[Bug c++/95003] coroutines: Wrong code for some reference capture cases.

2020-05-13 Thread iains at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95003

Iain Sandoe  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|NEW |RESOLVED

--- Comment #4 from Iain Sandoe  ---
so fixed for master and 10.2

[Bug c++/95003] coroutines: Wrong code for some reference capture cases.

2020-05-10 Thread cvs-commit at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95003

--- Comment #3 from CVS Commits  ---
The releases/gcc-10 branch has been updated by Iain D Sandoe
:

https://gcc.gnu.org/g:f09d898296c02d023ec51489fd233075ac553fd4

commit r10-8127-gf09d898296c02d023ec51489fd233075ac553fd4
Author: Iain Sandoe 
Date:   Sun May 10 14:20:13 2020 +0100

coroutines: Update TREE_SIDE_EFFECTS on inserted bind exprs.

There are several places where we insert bind expressions while
making the coroutine AST transforms.  These should be marked as
having side-effects where relevant, which had been omitted.  This
leads to at least one failure in the cppcoros test suite, where a loop
body is dropped in gimplification because it is not marked.

gcc/cp/ChangeLog:

2020-05-10  Iain Sandoe  

Backported from mainline
2020-05-08  Iain Sandoe  

PR c++/95003
* coroutines.cc (build_actor_fn): Ensure that bind scopes
are marked as having side-effects where necessary.
(replace_statement_captures): Likewise.
(morph_fn_to_coro): Likewise.

gcc/testsuite/ChangeLog:

2020-05-10  Iain Sandoe  

Backported from mainline
2020-05-08  Iain Sandoe  

PR c++/95003
* g++.dg/coroutines/torture/pr95003.C: New test.

[Bug c++/95003] coroutines: Wrong code for some reference capture cases.

2020-05-09 Thread iains at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95003

Iain Sandoe  changed:

   What|Removed |Added

 CC||lewissbaker.opensource@gmai
   ||l.com

--- Comment #2 from Iain Sandoe  ---
*** Bug 95017 has been marked as a duplicate of this bug. ***

[Bug c++/95003] coroutines: Wrong code for some reference capture cases.

2020-05-08 Thread cvs-commit at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95003

--- Comment #1 from CVS Commits  ---
The master branch has been updated by Iain D Sandoe :

https://gcc.gnu.org/g:234681eadf2c51d7b78270188d64601b7267330d

commit r11-208-g234681eadf2c51d7b78270188d64601b7267330d
Author: Iain Sandoe 
Date:   Sun May 3 14:20:13 2020 +0100

coroutines: Update TREE_SIDE_EFFECTS on inserted bind exprs.

There are several places where we insert bind expressions while
making the coroutine AST transforms.  These should be marked as
having side-effects where relevant, which had been omitted.  This
leads to at least one failure in the cppcoros test suite, where a loop
body is dropped in gimplification because it is not marked.

gcc/cp/ChangeLog:

2020-05-08  Iain Sandoe  

PR c++/95003
* coroutines.cc (build_actor_fn): Ensure that bind scopes
are marked as having side-effects where necessary.
(replace_statement_captures): Likewise.
(morph_fn_to_coro): Likewise.

gcc/testsuite/ChangeLog:

2020-05-08  Iain Sandoe  

PR c++/95003
* g++.dg/coroutines/torture/pr95003.C: New test.

[Bug c++/95003] coroutines: Wrong code for some reference capture cases.

2020-05-08 Thread iains at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95003

Iain Sandoe  changed:

   What|Removed |Added

 Ever confirmed|0   |1
   Keywords||wrong-code
   Target Milestone|--- |10.2
   Last reconfirmed||2020-05-08
   Assignee|unassigned at gcc dot gnu.org  |iains at gcc dot gnu.org
 Status|UNCONFIRMED |NEW