[Bug c++/58814] New: [4.7/4.8/4.9 Regression] ICE with volatile and attribute vector_size

[Bug c++/58814] New: [4.7/4.8/4.9 Regression] ICE with volatile and attribute vector_size

[reichelt at gcc dot gnu.org]

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

            Bug ID: 58814
           Summary: [4.7/4.8/4.9 Regression] ICE with volatile and
                    attribute vector_size
           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 valid code snippet triggers an ICE since GCC 4.7.0:

====================================================
volatile int i __attribute__((vector_size(8)));

void foo()
{
  i += i;
}
====================================================

bug.cc: In function 'void foo()':
bug.cc:5:5: internal compiler error: in stabilize_expr, at cp/tree.c:3769
   i += i;
     ^
0x6ef1fe stabilize_expr(tree_node*, tree_node**)
        ../../gcc/gcc/cp/tree.c:3769
0x68c9f4 cp_build_modify_expr(tree_node*, tree_code, tree_node*, int)
        ../../gcc/gcc/cp/typeck.c:7362
0x556885 build_new_op_1
        ../../gcc/gcc/cp/call.c:5438
0x556f57 build_new_op(unsigned int, tree_code, int, tree_node*, tree_node*,
tree_node*, tree_node**, int)
        ../../gcc/gcc/cp/call.c:5513
0x68e09d build_x_modify_expr(unsigned int, tree_node*, tree_code, tree_node*,
int)
        ../../gcc/gcc/cp/typeck.c:7534
0x651220 cp_parser_assignment_expression
        ../../gcc/gcc/cp/parser.c:8032
0x653523 cp_parser_expression
        ../../gcc/gcc/cp/parser.c:8158
0x653d2e cp_parser_expression
        ../../gcc/gcc/cp/parser.c:8197
0x653d2e cp_parser_expression_statement
        ../../gcc/gcc/cp/parser.c:9488
0x648fd7 cp_parser_statement
        ../../gcc/gcc/cp/parser.c:9339
0x64a2c2 cp_parser_statement_seq_opt
        ../../gcc/gcc/cp/parser.c:9611
0x64a406 cp_parser_compound_statement
        ../../gcc/gcc/cp/parser.c:9565
0x65da5b cp_parser_function_body
        ../../gcc/gcc/cp/parser.c:18408
0x65da5b cp_parser_ctor_initializer_opt_and_function_body
        ../../gcc/gcc/cp/parser.c:18444
0x661e5f cp_parser_function_definition_after_declarator
        ../../gcc/gcc/cp/parser.c:22452
0x662c32 cp_parser_function_definition_from_specifiers_and_declarator
        ../../gcc/gcc/cp/parser.c:22373
0x662c32 cp_parser_init_declarator
        ../../gcc/gcc/cp/parser.c:16419
0x663e7f cp_parser_simple_declaration
        ../../gcc/gcc/cp/parser.c:11054
0x6477e0 cp_parser_block_declaration
        ../../gcc/gcc/cp/parser.c:10935
0x66e9fe cp_parser_declaration
        ../../gcc/gcc/cp/parser.c:10832
Please submit a full bug report, [etc.]

[Bug c++/58814] [4.7/4.8/4.9 Regression] ICE with volatile and attribute vector_size

[reichelt at gcc dot gnu.org]

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

Volker Reichelt <reichelt at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |ice-on-valid-code
      Known to work|                            |4.5.0, 4.6.0
   Target Milestone|---                         |4.7.4
      Known to fail|                            |4.7.0, 4.8.0, 4.9.0

[Bug c++/58814] [4.7/4.8/4.9 Regression] ICE with volatile and attribute vector_size

[mpolacek at gcc dot gnu.org]

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

Marek Polacek <mpolacek at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2013-10-21
                 CC|                            |mpolacek at gcc dot gnu.org
     Ever confirmed|0                           |1

--- Comment #1 from Marek Polacek <mpolacek at gcc dot gnu.org> ---
Confirmed.

[Bug c++/58814] [4.7/4.8/4.9 Regression] ICE with volatile and attribute vector_size

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P2

[Bug c++/58814] [4.7/4.8/4.9 Regression] ICE with volatile and attribute vector_size

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |jason at gcc dot gnu.org
           Assignee|unassigned at gcc dot gnu.org      |jason at gcc dot gnu.org

[Bug c++/58814] [4.7/4.8/4.9 Regression] ICE with volatile and attribute vector_size

[jason at gcc dot gnu.org]

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

--- Comment #2 from Jason Merrill <jason at gcc dot gnu.org> ---
Author: jason
Date: Tue Jan 28 04:30:38 2014
New Revision: 207163

URL: http://gcc.gnu.org/viewcvs?rev=207163&root=gcc&view=rev
Log:
    PR c++/58814
    * typeck.c (cp_build_modify_expr): Make the RHS an rvalue before
    stabilizing.

Added:
    trunk/gcc/testsuite/g++.dg/ext/vector25.C
Modified:
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/typeck.c

[Bug c++/58814] [4.7/4.8/4.9 Regression] ICE with volatile and attribute vector_size

[jason at gcc dot gnu.org]

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

--- Comment #3 from Jason Merrill <jason at gcc dot gnu.org> ---
Author: jason
Date: Tue Jan 28 13:44:41 2014
New Revision: 207186

URL: http://gcc.gnu.org/viewcvs?rev=207186&root=gcc&view=rev
Log:
    PR c++/58814
    * typeck.c (cp_build_modify_expr): Make the RHS an rvalue before
    stabilizing.

Added:
    branches/gcc-4_8-branch/gcc/testsuite/g++.dg/ext/vector25.C
Modified:
    branches/gcc-4_8-branch/gcc/cp/ChangeLog
    branches/gcc-4_8-branch/gcc/cp/typeck.c

[Bug c++/58814] [4.7/4.8/4.9 Regression] ICE with volatile and attribute vector_size

[jason at gcc dot gnu.org]

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

Jason Merrill <jason at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
   Target Milestone|4.7.4                       |4.8.3

--- Comment #4 from Jason Merrill <jason at gcc dot gnu.org> ---
Fixed for 4.8.3/4.9.  Not fixing in 4.7.