[Bug c/97650] New: ICE: tree check: expected ssa_name, have addr_expr in vect_get_and_check_slp_defs, at tree-vect-slp.c:533 with "-Os -ftree-slp-vectorize -fallow-store-data-races"

[Bug c/97650] New: ICE: tree check: expected ssa_name, have addr_expr in vect_get_and_check_slp_defs, at tree-vect-slp.c:533 with "-Os -ftree-slp-vectorize -fallow-store-data-races"

[suochenyao at 163 dot com]

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

            Bug ID: 97650
           Summary: ICE: tree check: expected ssa_name, have addr_expr in
                    vect_get_and_check_slp_defs, at tree-vect-slp.c:533
                    with "-Os -ftree-slp-vectorize
                    -fallow-store-data-races"
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c
          Assignee: unassigned at gcc dot gnu.org
          Reporter: suochenyao at 163 dot com
  Target Milestone: ---

*******************************************************************************
OS and Platform:
CentOS Linux release 7.8.2003 (Core), x86_64 GNU/Linux
*******************************************************************************
Program:
short a=0;
unsigned long *volatile *volatile *volatile *b;
unsigned long *volatile *volatile *volatile **c[7];
void d() {
  short e=0;
  for (; a;) {
    e = 0;
    for (; e < 7; e++)
      c[e] = &b;
  }
}
int main() { return 0; }
*******************************************************************************
gcc version:
$ gcc -v
Using built-in specs.
COLLECT_GCC=/home/suocy/bin/gcc-dev/bin/gcc
COLLECT_LTO_WRAPPER=/home/suocy/bin/gcc-dev/libexec/gcc/x86_64-pc-linux-gnu/11.0.0/lto-wrapper
Target: x86_64-pc-linux-gnu
Configured with: ../configure --prefix=/home/suocy/bin/gcc-dev/
--disable-multilib --enable-languages=c,c++
Thread model: posix
Supported LTO compression algorithms: zlib
gcc version 11.0.0 20201030 (experimental) (GCC)
*******************************************************************************
Command Lines:
$ gcc -Wall -Wextra -fno-strict-aliasing -fwrapv -Os -ftree-slp-vectorize
-fallow-store-data-races a.c
during GIMPLE pass: slp
a.c: In function ‘d’:
a.c:4:6: internal compiler error: tree check: expected ssa_name, have addr_expr
in vect_get_and_check_slp_defs, at tree-vect-slp.c:533
    4 | void d() {
      |      ^
0x74bc24 tree_check_failed(tree_node const*, char const*, int, char const*,
...)
        ../../gcc/tree.c:9731
0x747607 tree_check(tree_node*, char const*, int, char const*, tree_code)
        ../../gcc/tree.h:3313
0x747607 vect_get_and_check_slp_defs
        ../../gcc/tree-vect-slp.c:533
0x1056988 vect_build_slp_tree_2
        ../../gcc/tree-vect-slp.c:1576
0x10565b9 vect_build_slp_tree
        ../../gcc/tree-vect-slp.c:1371
0x1056b9e vect_build_slp_tree_2
        ../../gcc/tree-vect-slp.c:1652
0x10565b9 vect_build_slp_tree
        ../../gcc/tree-vect-slp.c:1371
0x1056b9e vect_build_slp_tree_2
        ../../gcc/tree-vect-slp.c:1652
0x10565b9 vect_build_slp_tree
        ../../gcc/tree-vect-slp.c:1371
0x105933d vect_build_slp_instance
        ../../gcc/tree-vect-slp.c:2216
0x105ab8c vect_analyze_slp_instance
        ../../gcc/tree-vect-slp.c:2598
0x105b1b4 vect_analyze_slp(vec_info*, unsigned int)
        ../../gcc/tree-vect-slp.c:2626
0x106182f vect_slp_analyze_bb_1
        ../../gcc/tree-vect-slp.c:4139
0x106182f vect_slp_region
        ../../gcc/tree-vect-slp.c:4234
0x106182f vect_slp_bbs
        ../../gcc/tree-vect-slp.c:4382
0x1063364 vect_slp_function(function*)
        ../../gcc/tree-vect-slp.c:4468
0x1064e36 execute
        ../../gcc/tree-vectorizer.c:1437
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.

[Bug tree-optimization/97650] ICE: tree check: expected ssa_name, have addr_expr in vect_get_and_check_slp_defs, at tree-vect-slp.c:533 with "-Os -ftree-slp-vectorize -fallow-store-data-races"

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |ASSIGNED
          Component|c                           |tree-optimization
   Last reconfirmed|                            |2020-11-02
           Assignee|unassigned at gcc dot gnu.org      |rguenth at gcc dot gnu.org

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
Mine.

[Bug tree-optimization/97650] ICE: tree check: expected ssa_name, have addr_expr in vect_get_and_check_slp_defs, at tree-vect-slp.c:533 with "-Os -ftree-slp-vectorize -fallow-store-data-races"

[marxin at gcc dot gnu.org]

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

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |marxin at gcc dot gnu.org

--- Comment #2 from Martin Liška <marxin at gcc dot gnu.org> ---
Just for the record, started with r11-4428-g4a369d199bf2f34e.

[Bug tree-optimization/97650] ICE: tree check: expected ssa_name, have addr_expr in vect_get_and_check_slp_defs, at tree-vect-slp.c:533 with "-Os -ftree-slp-vectorize -fallow-store-data-races"

[cvs-commit at gcc dot gnu.org]

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

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Richard Biener <rguenth@gcc.gnu.org>:

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

commit r11-4602-ga159081ad4259c42456bd3dc2b637747c373c5a5
Author: Richard Biener <rguenther@suse.de>
Date:   Mon Nov 2 08:59:02 2020 +0100

    tree-optimization/97650 - fix ICE in vect_get_and_check_slp_defs

    I was mistaken to treat vect_external_def as only applying to
    SSA_NAME defs, so check for that.

    2020-11-02  Richard Biener  <rguenther@suse.de>

            PR tree-optimization/97650
            * tree-vect-slp.c (vect_get_and_check_slp_defs): Check
            for SSA_NAME before checking SSA_NAME_IS_DEFAULT_DEF.

            * gcc.dg/vect/bb-slp-pr97650.c: New testcase.

[Bug tree-optimization/97650] ICE: tree check: expected ssa_name, have addr_expr in vect_get_and_check_slp_defs, at tree-vect-slp.c:533 with "-Os -ftree-slp-vectorize -fallow-store-data-races"

[rguenth at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

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