public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/99727] New: [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3
@ 2021-03-23 12:45 acoplan at gcc dot gnu.org
  2021-03-23 12:45 ` [Bug target/99727] " acoplan at gcc dot gnu.org
                   ` (6 more replies)
  0 siblings, 7 replies; 8+ messages in thread
From: acoplan at gcc dot gnu.org @ 2021-03-23 12:45 UTC (permalink / raw)
  To: gcc-bugs

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

            Bug ID: 99727
           Summary: [11 Regression] MVE: ICE (segfault) in
                    arm_print_operand at -O3
           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
char a[7][25][15][14];
void b() {
  for (int c;; ++c)
    for (int d = 0; d < 25; ++d)
      for (int e = 0; e < 15; ++e)
        for (int f = 0; f < 14; ++f)
          a[c][d][e][f] = 1;
}
$ arm-eabi-gcc -c test.c -march=armv8.1-m.main+mve -mfloat-abi=hard -O3
during RTL pass: final
test.c: In function 'b':
test.c:8:1: internal compiler error: Segmentation fault
    8 | }
      | ^
0xd41821 crash_signal
        /home/alecop01/toolchain/src/gcc/gcc/toplev.c:327
0x118eba5 arm_print_operand
        /home/alecop01/toolchain/src/gcc/gcc/config/arm/arm.c:24201
0x960465 output_operand(rtx_def*, int)
        /home/alecop01/toolchain/src/gcc/gcc/final.c:4069
0x960f44 output_asm_insn(char const*, rtx_def**)
        /home/alecop01/toolchain/src/gcc/gcc/final.c:3962
0x9639f5 final_scan_insn_1
        /home/alecop01/toolchain/src/gcc/gcc/final.c:3125
0x963c8d final_scan_insn(rtx_insn*, _IO_FILE*, int, int, int*)
        /home/alecop01/toolchain/src/gcc/gcc/final.c:3171
0x96489a final_1
        /home/alecop01/toolchain/src/gcc/gcc/final.c:2022
0x964bf8 rest_of_handle_final
        /home/alecop01/toolchain/src/gcc/gcc/final.c:4676
0x964bf8 execute
        /home/alecop01/toolchain/src/gcc/gcc/final.c:4754
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.

I can't reproduce it on the GCC 10 branch.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/99727] [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3
  2021-03-23 12:45 [Bug target/99727] New: [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 acoplan at gcc dot gnu.org
@ 2021-03-23 12:45 ` acoplan at gcc dot gnu.org
  2021-03-23 14:16 ` [Bug target/99727] [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 since r11-6616-g25bef689 acoplan at gcc dot gnu.org
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: acoplan at gcc dot gnu.org @ 2021-03-23 12:45 UTC (permalink / raw)
  To: gcc-bugs

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |11.0
      Known to work|                            |10.2.1
             Target|                            |arm
   Target Milestone|---                         |11.0
           Keywords|                            |ice-on-valid-code

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/99727] [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 since r11-6616-g25bef689
  2021-03-23 12:45 [Bug target/99727] New: [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 acoplan at gcc dot gnu.org
  2021-03-23 12:45 ` [Bug target/99727] " acoplan at gcc dot gnu.org
@ 2021-03-23 14:16 ` acoplan at gcc dot gnu.org
  2021-03-23 21:55 ` clyon at gcc dot gnu.org
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: acoplan at gcc dot gnu.org @ 2021-03-23 14:16 UTC (permalink / raw)
  To: gcc-bugs

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |clyon at gcc dot gnu.org
            Summary|[11 Regression] MVE: ICE    |[11 Regression] MVE: ICE
                   |(segfault) in               |(segfault) in
                   |arm_print_operand at -O3    |arm_print_operand at -O3
                   |                            |since r11-6616-g25bef689

--- Comment #1 from Alex Coplan <acoplan at gcc dot gnu.org> ---
Started with r11-6616-g25bef68902f42f414f99626cefb2d3df81de7dc8:

commit 25bef68902f42f414f99626cefb2d3df81de7dc8 (refs/bisect/bad)
Author: Christophe Lyon <christophe.lyon@linaro.org>
Date:   Tue Jan 12 16:47:27 2021

    arm: Add movmisalign patterns for MVE (PR target/97875)

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/99727] [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 since r11-6616-g25bef689
  2021-03-23 12:45 [Bug target/99727] New: [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 acoplan at gcc dot gnu.org
  2021-03-23 12:45 ` [Bug target/99727] " acoplan at gcc dot gnu.org
  2021-03-23 14:16 ` [Bug target/99727] [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 since r11-6616-g25bef689 acoplan at gcc dot gnu.org
@ 2021-03-23 21:55 ` clyon at gcc dot gnu.org
  2021-03-24  8:32 ` rguenth at gcc dot gnu.org
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: clyon at gcc dot gnu.org @ 2021-03-23 21:55 UTC (permalink / raw)
  To: gcc-bugs

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

Christophe Lyon <clyon at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2021-03-23
     Ever confirmed|0                           |1

--- Comment #2 from Christophe Lyon <clyon at gcc dot gnu.org> ---
Looks like a constraint problem: I kept the Um constraint as used by Neon,
while MVE needs Ux.

This patch fixes the ICE:
diff --git a/gcc/config/arm/mve.md b/gcc/config/arm/mve.md
index 440fd6a..1351863 100644
--- a/gcc/config/arm/mve.md
+++ b/gcc/config/arm/mve.md
@@ -10858,7 +10858,7 @@ (define_insn "arm_vcx3q<a>_p_v16qi"
 )

 (define_insn "*movmisalign<mode>_mve_store"
-  [(set (match_operand:MVE_VLD_ST 0 "neon_permissive_struct_operand"       
"=Um")
+  [(set (match_operand:MVE_VLD_ST 0 "neon_permissive_struct_operand"       
"=Ux")
        (unspec:MVE_VLD_ST [(match_operand:MVE_VLD_ST 1 "s_register_operand" "
w")]
         UNSPEC_MISALIGNED_ACCESS))]
   "((TARGET_HAVE_MVE && VALID_MVE_SI_MODE (<MODE>mode))
@@ -10871,7 +10871,7 @@ (define_insn "*movmisalign<mode>_mve_store"

 (define_insn "*movmisalign<mode>_mve_load"
   [(set (match_operand:MVE_VLD_ST 0 "s_register_operand"                      
         "=w")
-       (unspec:MVE_VLD_ST [(match_operand:MVE_VLD_ST 1
"neon_permissive_struct_operand" " Um")]
+       (unspec:MVE_VLD_ST [(match_operand:MVE_VLD_ST 1
"neon_permissive_struct_operand" " Ux")]
         UNSPEC_MISALIGNED_ACCESS))]
   "((TARGET_HAVE_MVE && VALID_MVE_SI_MODE (<MODE>mode))
     || (TARGET_HAVE_MVE_FLOAT && VALID_MVE_SF_MODE (<MODE>mode)))

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/99727] [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 since r11-6616-g25bef689
  2021-03-23 12:45 [Bug target/99727] New: [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 acoplan at gcc dot gnu.org
                   ` (2 preceding siblings ...)
  2021-03-23 21:55 ` clyon at gcc dot gnu.org
@ 2021-03-24  8:32 ` rguenth at gcc dot gnu.org
  2021-03-24 14:23 ` cvs-commit at gcc dot gnu.org
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: rguenth at gcc dot gnu.org @ 2021-03-24  8:32 UTC (permalink / raw)
  To: gcc-bugs

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/99727] [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 since r11-6616-g25bef689
  2021-03-23 12:45 [Bug target/99727] New: [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 acoplan at gcc dot gnu.org
                   ` (3 preceding siblings ...)
  2021-03-24  8:32 ` rguenth at gcc dot gnu.org
@ 2021-03-24 14:23 ` cvs-commit at gcc dot gnu.org
  2021-03-24 14:24 ` clyon at gcc dot gnu.org
  2021-03-24 14:28 ` acoplan at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2021-03-24 14:23 UTC (permalink / raw)
  To: gcc-bugs

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

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

https://gcc.gnu.org/g:79cdbabbc4cbf79491c274220928406dff3d4039

commit r11-7816-g79cdbabbc4cbf79491c274220928406dff3d4039
Author: Christophe Lyon <christophe.lyon@linaro.org>
Date:   Wed Mar 24 08:51:41 2021 +0000

    arm: Fix MVE constraints for movmisalign [PR target/99727]

    MVE has different constraints than Neon for load/store: we should use
    the Ux constraint instead of Um.

    2021-03-24  Christophe Lyon  <christophe.lyon@linaro.org>

            PR target/99727
            gcc/
            * config/arm/mve.md (movmisalign<mode>_mve_store): Use Ux
            constraint.
            (movmisalign<mode>_mve_load): Likewise.

            gcc/testsuite/
            * gcc.target/arm/pr99727.c: New test.

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/99727] [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 since r11-6616-g25bef689
  2021-03-23 12:45 [Bug target/99727] New: [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 acoplan at gcc dot gnu.org
                   ` (4 preceding siblings ...)
  2021-03-24 14:23 ` cvs-commit at gcc dot gnu.org
@ 2021-03-24 14:24 ` clyon at gcc dot gnu.org
  2021-03-24 14:28 ` acoplan at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: clyon at gcc dot gnu.org @ 2021-03-24 14:24 UTC (permalink / raw)
  To: gcc-bugs

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

Christophe Lyon <clyon at gcc dot gnu.org> changed:

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

--- Comment #4 from Christophe Lyon <clyon at gcc dot gnu.org> ---
Fixed

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Bug target/99727] [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 since r11-6616-g25bef689
  2021-03-23 12:45 [Bug target/99727] New: [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 acoplan at gcc dot gnu.org
                   ` (5 preceding siblings ...)
  2021-03-24 14:24 ` clyon at gcc dot gnu.org
@ 2021-03-24 14:28 ` acoplan at gcc dot gnu.org
  6 siblings, 0 replies; 8+ messages in thread
From: acoplan at gcc dot gnu.org @ 2021-03-24 14:28 UTC (permalink / raw)
  To: gcc-bugs

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

--- Comment #5 from Alex Coplan <acoplan at gcc dot gnu.org> ---
Thanks for the quick fix :)

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2021-03-24 14:28 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-23 12:45 [Bug target/99727] New: [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 acoplan at gcc dot gnu.org
2021-03-23 12:45 ` [Bug target/99727] " acoplan at gcc dot gnu.org
2021-03-23 14:16 ` [Bug target/99727] [11 Regression] MVE: ICE (segfault) in arm_print_operand at -O3 since r11-6616-g25bef689 acoplan at gcc dot gnu.org
2021-03-23 21:55 ` clyon at gcc dot gnu.org
2021-03-24  8:32 ` rguenth at gcc dot gnu.org
2021-03-24 14:23 ` cvs-commit at gcc dot gnu.org
2021-03-24 14:24 ` clyon at gcc dot gnu.org
2021-03-24 14:28 ` acoplan at gcc dot gnu.org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).