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

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

[reichelt at gcc dot gnu.org]

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

            Bug ID: 60189
           Summary: ICE with invalid use of _Cilk_sync
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: reichelt at gcc dot gnu.org

The following invalid code snippet (compiled with "-fcilkplus")
triggers an ICE on trunk:

=======================
void foo()
{
  _Cilk_sync return;
}
=======================

bug.cc: In function 'void foo()':
bug.cc:5:3: internal compiler error: Segmentation fault
   _Cilk_sync return;
   ^
0xba523f crash_signal
        ../../gcc/gcc/toplev.c:337
0x6b9008 cp_parser_expression_statement
        ../../gcc/gcc/cp/parser.c:9635
0x6ae398 cp_parser_statement
        ../../gcc/gcc/cp/parser.c:9473
0x6af169 cp_parser_statement_seq_opt
        ../../gcc/gcc/cp/parser.c:9745
0x6af2d6 cp_parser_compound_statement
        ../../gcc/gcc/cp/parser.c:9699
0x6c34ab cp_parser_function_body
        ../../gcc/gcc/cp/parser.c:18641
0x6c34ab cp_parser_ctor_initializer_opt_and_function_body
        ../../gcc/gcc/cp/parser.c:18677
0x6c77f2 cp_parser_function_definition_after_declarator
        ../../gcc/gcc/cp/parser.c:22792
0x6c8691 cp_parser_function_definition_from_specifiers_and_declarator
        ../../gcc/gcc/cp/parser.c:22704
0x6c8691 cp_parser_init_declarator
        ../../gcc/gcc/cp/parser.c:16585
0x6c9929 cp_parser_simple_declaration
        ../../gcc/gcc/cp/parser.c:11205
0x6acc13 cp_parser_block_declaration
        ../../gcc/gcc/cp/parser.c:11086
0x6d3ec2 cp_parser_declaration
        ../../gcc/gcc/cp/parser.c:10983
0x6d2bb8 cp_parser_declaration_seq_opt
        ../../gcc/gcc/cp/parser.c:10869
0x6d445a cp_parser_translation_unit
        ../../gcc/gcc/cp/parser.c:4014
0x6d445a c_parse_file()
        ../../gcc/gcc/cp/parser.c:31536
0x7f37c3 c_common_parse_file()
        ../../gcc/gcc/c-family/c-opts.c:1060
Please submit a full bug report, [etc.]

[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 <reichelt at gcc dot gnu.org> 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 <reichelt at gcc dot gnu.org> ---
Balaji, this looks like your domain, would you mind having a look?

[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 <izamyatin at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |izamyatin at gmail dot com

--- Comment #2 from Igor Zamyatin <izamyatin at gmail dot com> ---
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’
>From gcc-bugs-return-445350-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Tue Mar 04 08:31:21 2014
Return-Path: <gcc-bugs-return-445350-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 8179 invoked by alias); 4 Mar 2014 08:31:20 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Delivered-To: mailing list gcc-bugs@gcc.gnu.org
Received: (qmail 8126 invoked by uid 48); 4 Mar 2014 08:31:16 -0000
From: "izamyatin at gmail dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c++/60082] Certain Cilk keywords executable Hanging for -O1
Date: Tue, 04 Mar 2014 08:31:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: c++
X-Bugzilla-Version: 4.9.0
X-Bugzilla-Keywords:
X-Bugzilla-Severity: major
X-Bugzilla-Who: izamyatin at gmail dot com
X-Bugzilla-Status: NEW
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields: cc
Message-ID: <bug-60082-4-6x3zdLGrfr@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-60082-4@http.gcc.gnu.org/bugzilla/>
References: <bug-60082-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2014-03/txt/msg00219.txt.bz2
Content-length: 402

http://gcc.gnu.org/bugzilla/show_bug.cgi?id`082

Igor Zamyatin <izamyatin at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |izamyatin at gmail dot com

--- Comment #6 from Igor Zamyatin <izamyatin at gmail dot com> ---
Fixed after r207623?

[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 <izamyatin at gmail dot com> ---
Ah, g++ gives the ICE :(

[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 <kyukhin at gcc dot gnu.org> ---
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
>From gcc-bugs-return-452129-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Wed May 21 08:04:43 2014
Return-Path: <gcc-bugs-return-452129-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 28763 invoked by alias); 21 May 2014 08:04:42 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Delivered-To: mailing list gcc-bugs@gcc.gnu.org
Received: (qmail 28723 invoked by uid 55); 21 May 2014 08:04:39 -0000
From: "jakub at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug middle-end/61252] Invalid code produced for omp  simd reduction(min:var) where var is reference
Date: Wed, 21 May 2014 08:04:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: middle-end
X-Bugzilla-Version: 4.9.1
X-Bugzilla-Keywords:
X-Bugzilla-Severity: normal
X-Bugzilla-Who: jakub at gcc dot gnu.org
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: jakub at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields:
Message-ID: <bug-61252-4-v8zhwZALCA@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-61252-4@http.gcc.gnu.org/bugzilla/>
References: <bug-61252-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2014-05/txt/msg01821.txt.bz2
Content-length: 688

https://gcc.gnu.org/bugzilla/show_bug.cgi?ida252

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Wed May 21 08:04:03 2014
New Revision: 210679

URL: http://gcc.gnu.org/viewcvs?rev!0679&root=gcc&view=rev
Log:
    PR middle-end/61252
    * omp-low.c (handle_simd_reference): New function.
    (lower_rec_input_clauses): Use it.  Defer adding reference
    initialization even for reduction without placeholder if in simd,
    handle it properly later on.

    * testsuite/libgomp.c++/simd-9.C: New test.

Added:
    trunk/libgomp/testsuite/libgomp.c++/simd-9.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/omp-low.c
    trunk/libgomp/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 #5 from Kirill Yukhin <kyukhin at gcc dot gnu.org> ---
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

[reichelt at gcc dot gnu.org]

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

Volker Reichelt <reichelt at gcc dot gnu.org> 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 <reichelt at gcc dot gnu.org> ---
Fixed for GCC 4.9.1 by Igor's patch.