[Bug target/99820] New: aarch64: ICE (segfault) in aarch64_analyze_loop_vinfo with -moverride=tune=use_new_vector_costs

[Bug target/99820] New: aarch64: ICE (segfault) in aarch64_analyze_loop_vinfo with -moverride=tune=use_new_vector_costs

[acoplan at gcc dot gnu.org]

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

            Bug ID: 99820
           Summary: aarch64: ICE (segfault) in aarch64_analyze_loop_vinfo
                    with -moverride=tune=use_new_vector_costs
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: acoplan at gcc dot gnu.org
  Target Milestone: ---

The following fails:

$ cat test.c
extern short a[];
void f() {
  for (int i = 0; i < 100; i++)
    a[i] = 0;
}
$ aarch64-linux-gnu-gcc -c test.c -march=armv8.2-a+sve -O -ftree-vectorize
-moverride=tune=use_new_vector_costs
during GIMPLE pass: vect
test.c: In function ‘f’:
test.c:2:6: internal compiler error: Segmentation fault
    2 | void f() {
      |      ^
0xdf6c93 crash_signal
        /home/alecop01/toolchain/src/gcc/gcc/toplev.c:327
0x1252b9b aarch64_analyze_loop_vinfo
        /home/alecop01/toolchain/src/gcc/gcc/config/aarch64/aarch64.c:14463
0x1252b9b aarch64_add_stmt_cost
        /home/alecop01/toolchain/src/gcc/gcc/config/aarch64/aarch64.c:15292
0x111fdd3 add_stmt_cost
        /home/alecop01/toolchain/src/gcc/gcc/tree-vectorizer.h:1477
0x111fdd3 vect_compute_single_scalar_iteration_cost
        /home/alecop01/toolchain/src/gcc/gcc/tree-vect-loop.c:1292
0x111fdd3 vect_analyze_loop_2
        /home/alecop01/toolchain/src/gcc/gcc/tree-vect-loop.c:2311
0x1122169 vect_analyze_loop(loop*, vec_info_shared*)
        /home/alecop01/toolchain/src/gcc/gcc/tree-vect-loop.c:2985
0x115847b try_vectorize_loop_1
        /home/alecop01/toolchain/src/gcc/gcc/tree-vectorizer.c:1009
0x11589b3 try_vectorize_loop
        /home/alecop01/toolchain/src/gcc/gcc/tree-vectorizer.c:1162
0x1158cf9 vectorize_loops()
        /home/alecop01/toolchain/src/gcc/gcc/tree-vectorizer.c:1243
0xfd74cb execute
        /home/alecop01/toolchain/src/gcc/gcc/tree-ssa-loop.c:414
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 target/99820] aarch64: ICE (segfault) in aarch64_analyze_loop_vinfo with -moverride=tune=use_new_vector_costs

[acoplan at gcc dot gnu.org]

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

Alex Coplan <acoplan at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |11.0
   Target Milestone|---                         |11.0
           Keywords|                            |ice-on-valid-code
                 CC|                            |rsandifo at gcc dot gnu.org
             Target|                            |aarch64

[Bug target/99820] aarch64: ICE (segfault) in aarch64_analyze_loop_vinfo with -moverride=tune=use_new_vector_costs

[cvs-commit at gcc dot gnu.org]

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

--- Comment #1 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Kyrylo Tkachov <ktkachov@gcc.gnu.org>:

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

commit r11-7913-gc277abd9cd3d10db59f9965d7d6356868da42a9f
Author: Kyrylo Tkachov <kyrylo.tkachov@arm.com>
Date:   Tue Mar 30 16:42:17 2021 +0100

    aarch64: PR target/99820: Guard on available SVE issue info before using

    This fixes a simple segfault ICE when using the use_new_vector_costs
tunable with a CPU tuning that it wasn't intended for.
    I'm not adding a testcase here as we intend to remove the tunable for GCC
12 anyway (the new costing logic will remain and will benefit
    from this extra check, but the -moverride option will no longer exist).

    gcc/ChangeLog:

            PR target/99820
            * config/aarch64/aarch64.c (aarch64_analyze_loop_vinfo): Check for
            available issue_info before using it.

[Bug target/99820] aarch64: ICE (segfault) in aarch64_analyze_loop_vinfo with -moverride=tune=use_new_vector_costs

[ktkachov at gcc dot gnu.org]

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

ktkachov at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ktkachov at gcc dot gnu.org
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from ktkachov at gcc dot gnu.org ---
Fixed.