public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c++/98935] New: [coroutines] co_await on statement expressions causes ICE
@ 2021-02-02 15:47 mail+gnu at tzik dot jp
  2021-09-27 21:09 ` [Bug c++/98935] " delleyves at gmx dot ch
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: mail+gnu at tzik dot jp @ 2021-02-02 15:47 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98935

            Bug ID: 98935
           Summary: [coroutines] co_await on statement expressions causes
                    ICE
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: mail+gnu at tzik dot jp
  Target Milestone: ---

In a repro case below, `co_await ({auto x = future{}; x;});` causes an internal
compiler error.
https://wandbox.org/permlink/3jX83p34mVDfBgc1

The compile command was:
$ g++ -std=c++20 failcase.cc
where g++ was the current trunk (11.0.0).
---- failcase.cc
#if __has_include(<coroutine>)
#include <coroutine>
#elif __has_include(<experimental/coroutine>)
#include <experimental/coroutine>
namespace std {
using namespace std::experimental;
}
#endif

struct future {
  struct promise_type {
    void return_void() {}
    std::suspend_always initial_suspend() noexcept { return {}; }
    std::suspend_always final_suspend() noexcept { return {}; }
    void unhandled_exception() {}
    future get_return_object() { return {}; }
  };
  bool await_ready() { return false; }
  void await_suspend(std::coroutine_handle<>) {}
  void await_resume() {}
};

future failcase() {
  co_await ({auto x = future{}; x;});
}
----

And its backtrace was:
----
prog.cc: In function 'future failcase()':
prog.cc:25:1: internal compiler error: Segmentation fault
   25 | }
      | ^
0xca67ef crash_signal
        ../../source/gcc/toplev.c:327
0xf1866b walk_tree_1(tree_node**, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*,
tree_node* (*)(tree_node**, int*, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*))
        ../../source/gcc/tree.c:12105
0xf18c82 walk_tree_1(tree_node**, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*,
tree_node* (*)(tree_node**, int*, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*))
        ../../source/gcc/tree.c:12340
0xf18ad1 walk_tree_1(tree_node**, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*,
tree_node* (*)(tree_node**, int*, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*))
        ../../source/gcc/tree.c:12204
0x680293 transform_local_var_uses
        ../../source/gcc/cp/coroutines.cc:1903
0xf1865b walk_tree_1(tree_node**, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*,
tree_node* (*)(tree_node**, int*, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*))
        ../../source/gcc/tree.c:12099
0x680293 transform_local_var_uses
        ../../source/gcc/cp/coroutines.cc:1903
0xf1865b walk_tree_1(tree_node**, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*,
tree_node* (*)(tree_node**, int*, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*))
        ../../source/gcc/tree.c:12099
0xf18ad1 walk_tree_1(tree_node**, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*,
tree_node* (*)(tree_node**, int*, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*))
        ../../source/gcc/tree.c:12204
0xf187aa walk_tree_1(tree_node**, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*,
tree_node* (*)(tree_node**, int*, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*))
        ../../source/gcc/tree.c:12436
0xf18ad1 walk_tree_1(tree_node**, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*,
tree_node* (*)(tree_node**, int*, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*))
        ../../source/gcc/tree.c:12204
0x680293 transform_local_var_uses
        ../../source/gcc/cp/coroutines.cc:1903
0xf1865b walk_tree_1(tree_node**, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*,
tree_node* (*)(tree_node**, int*, tree_node* (*)(tree_node**, int*, void*),
void*, hash_set<tree_node*, false, default_hash_traits<tree_node*> >*))
        ../../source/gcc/tree.c:12099
0x680a27 build_actor_fn
        ../../source/gcc/cp/coroutines.cc:2085
0x6832a5 morph_fn_to_coro(tree_node*, tree_node**, tree_node**)
        ../../source/gcc/cp/coroutines.cc:4750
0x6b4f87 finish_function(bool)
        ../../source/gcc/cp/decl.c:17155
0x76da42 cp_parser_function_definition_after_declarator
        ../../source/gcc/cp/parser.c:29941
0x76eb9c cp_parser_function_definition_from_specifiers_and_declarator
        ../../source/gcc/cp/parser.c:29854
0x76eb9c cp_parser_init_declarator
        ../../source/gcc/cp/parser.c:21564
0x74d5b4 cp_parser_simple_declaration
        ../../source/gcc/cp/parser.c:14381
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See <https://gcc.gnu.org/bugs/> for instructions.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug c++/98935] [coroutines] co_await on statement expressions causes ICE
  2021-02-02 15:47 [Bug c++/98935] New: [coroutines] co_await on statement expressions causes ICE mail+gnu at tzik dot jp
@ 2021-09-27 21:09 ` delleyves at gmx dot ch
  2021-10-01 19:54 ` iains at gcc dot gnu.org
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: delleyves at gmx dot ch @ 2021-09-27 21:09 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98935

--- Comment #1 from Yves Delley <delleyves at gmx dot ch> ---
The provided fail-case seems to compile fine on godbolt.org 10.2, but started
to ICE with 10.3, as well as 11.1 and 11.2.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug c++/98935] [coroutines] co_await on statement expressions causes ICE
  2021-02-02 15:47 [Bug c++/98935] New: [coroutines] co_await on statement expressions causes ICE mail+gnu at tzik dot jp
  2021-09-27 21:09 ` [Bug c++/98935] " delleyves at gmx dot ch
@ 2021-10-01 19:54 ` iains at gcc dot gnu.org
  2022-06-28 10:43 ` jakub at gcc dot gnu.org
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: iains at gcc dot gnu.org @ 2021-10-01 19:54 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98935

Iain Sandoe <iains at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |10.4
     Ever confirmed|0                           |1
   Last reconfirmed|                            |2021-10-01
             Status|UNCONFIRMED                 |NEW

--- Comment #2 from Iain Sandoe <iains at gcc dot gnu.org> ---
statement expressions have never been handled explicitly (deferred since they
are an extension) .. however, it is planned to support them,

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug c++/98935] [coroutines] co_await on statement expressions causes ICE
  2021-02-02 15:47 [Bug c++/98935] New: [coroutines] co_await on statement expressions causes ICE mail+gnu at tzik dot jp
  2021-09-27 21:09 ` [Bug c++/98935] " delleyves at gmx dot ch
  2021-10-01 19:54 ` iains at gcc dot gnu.org
@ 2022-06-28 10:43 ` jakub at gcc dot gnu.org
  2023-07-07  9:24 ` rguenth at gcc dot gnu.org
  2024-02-01 21:07 ` lozko.roma at gmail dot com
  4 siblings, 0 replies; 6+ messages in thread
From: jakub at gcc dot gnu.org @ 2022-06-28 10:43 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98935

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|10.4                        |10.5

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 10.4 is being released, retargeting bugs to GCC 10.5.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug c++/98935] [coroutines] co_await on statement expressions causes ICE
  2021-02-02 15:47 [Bug c++/98935] New: [coroutines] co_await on statement expressions causes ICE mail+gnu at tzik dot jp
                   ` (2 preceding siblings ...)
  2022-06-28 10:43 ` jakub at gcc dot gnu.org
@ 2023-07-07  9:24 ` rguenth at gcc dot gnu.org
  2024-02-01 21:07 ` lozko.roma at gmail dot com
  4 siblings, 0 replies; 6+ messages in thread
From: rguenth at gcc dot gnu.org @ 2023-07-07  9:24 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98935

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to work|                            |11.3.0
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED
      Known to fail|                            |10.5.0, 11.2.0
   Target Milestone|10.5                        |11.3

--- Comment #4 from Richard Biener <rguenth at gcc dot gnu.org> ---
Fixed in GCC 11.3.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Bug c++/98935] [coroutines] co_await on statement expressions causes ICE
  2021-02-02 15:47 [Bug c++/98935] New: [coroutines] co_await on statement expressions causes ICE mail+gnu at tzik dot jp
                   ` (3 preceding siblings ...)
  2023-07-07  9:24 ` rguenth at gcc dot gnu.org
@ 2024-02-01 21:07 ` lozko.roma at gmail dot com
  4 siblings, 0 replies; 6+ messages in thread
From: lozko.roma at gmail dot com @ 2024-02-01 21:07 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98935

Roman Lozko <lozko.roma at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |lozko.roma at gmail dot com

--- Comment #5 from Roman Lozko <lozko.roma at gmail dot com> ---
Broken in trunk
internal compiler error: in register_local_var_uses, at cp/coroutines.cc:3903
https://godbolt.org/z/j9KTsroxj

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2024-02-01 21:07 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-02 15:47 [Bug c++/98935] New: [coroutines] co_await on statement expressions causes ICE mail+gnu at tzik dot jp
2021-09-27 21:09 ` [Bug c++/98935] " delleyves at gmx dot ch
2021-10-01 19:54 ` iains at gcc dot gnu.org
2022-06-28 10:43 ` jakub at gcc dot gnu.org
2023-07-07  9:24 ` rguenth at gcc dot gnu.org
2024-02-01 21:07 ` lozko.roma at gmail dot com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).