[Bug c++/60189] ICE with invalid use of _Cilk_sync

[reichelt at gcc dot gnu.org]

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

Volker Reichelt  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
  Known to work||4.9.1
 Resolution|--- |FIXED
   Target Milestone|--- |4.9.1

--- Comment #6 from Volker Reichelt  ---
Fixed for GCC 4.9.1 by Igor's patch.

[Bug c++/60189] ICE with invalid use of _Cilk_sync

[kyukhin at gcc dot gnu.org]

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

--- Comment #5 from Kirill Yukhin  ---
Author: kyukhin
Date: Wed May 21 08:05:42 2014
New Revision: 210680

URL: http://gcc.gnu.org/viewcvs?rev=210680&root=gcc&view=rev
Log:
gcc/cp
PR c/60189
* parser.c (cp_parser_postfix_expression): Move handling of cilk_sync
from here to...
(cp_parser_statement): ...here. Make sure only semicolon can go after
Cilk_sync.

gcc/testsuite
PR c++/60189
* c-c++-common/cilk-plus/CK/invalid_sync.cc: New test.



Added:
   
branches/gcc-4_9-branch/gcc/testsuite/c-c++-common/cilk-plus/CK/invalid_sync.cc
Modified:
branches/gcc-4_9-branch/gcc/cp/ChangeLog
branches/gcc-4_9-branch/gcc/cp/parser.c
branches/gcc-4_9-branch/gcc/testsuite/ChangeLog

[Bug c++/60189] ICE with invalid use of _Cilk_sync

[kyukhin at gcc dot gnu.org]

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

--- Comment #4 from Kirill Yukhin  ---
Author: kyukhin
Date: Wed May 21 08:03:06 2014
New Revision: 210678

URL: http://gcc.gnu.org/viewcvs?rev=210678&root=gcc&view=rev
Log:
gcc/cp
PR c/60189
* parser.c (cp_parser_postfix_expression): Move handling of cilk_sync
from here to...
(cp_parser_statement): ...here. Make sure only semicolon can go after
Cilk_sync.

gcc/testsuite
PR c++/60189
* c-c++-common/cilk-plus/CK/invalid_sync.cÑ: New test.



Added:
trunk/gcc/testsuite/c-c++-common/cilk-plus/CK/invalid_sync.cc
Modified:
trunk/gcc/cp/ChangeLog
trunk/gcc/cp/parser.c
trunk/gcc/testsuite/ChangeLog

[Bug c++/60189] ICE with invalid use of _Cilk_sync

[izamyatin at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60189

--- Comment #3 from Igor Zamyatin  ---
Ah, g++ gives the ICE :(

[Bug c++/60189] ICE with invalid use of _Cilk_sync

[izamyatin at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60189

Igor Zamyatin  changed:

   What|Removed |Added

 CC||izamyatin at gmail dot com

--- Comment #2 from Igor Zamyatin  ---
Now on trunk I see the following

cilk_test_60189.c: In function ‘foo’:
cilk_test_60189.c:3:16: error: expected ‘;’ before ‘return’
 _Cilk_sync return;
^
cilk_test_60189.c:3:16: error: expected ‘_Cilk_spawn’ before
‘_Cilk_sync’

[Bug c++/60189] ICE with invalid use of _Cilk_sync

[reichelt at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60189

Volker Reichelt  changed:

   What|Removed |Added

   Keywords||ice-on-invalid-code
 CC||bviyer at gcc dot gnu.org
  Known to fail||4.9.0

--- Comment #1 from Volker Reichelt  ---
Balaji, this looks like your domain, would you mind having a look?