[Bug target/83845] [8 regression] new failures after r256620
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83845 Richard Biener changed: What|Removed |Added Target Milestone|--- |8.0
[Bug target/83845] [8 regression] new failures after r256620
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83845 Richard Biener changed: What|Removed |Added Priority|P3 |P1
[Bug target/83845] [8 regression] new failures after r256620
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83845 nsz at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed||2018-01-17 CC||nsz at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from nsz at gcc dot gnu.org --- on aarch64_be-none-elf details for FAIL: gcc.target/aarch64/sve/mask_struct_load_3.c -march=armv8.2-a+sve (internal compiler error) spawn -ignore SIGHUP /B/gcc/xgcc -B/B/gcc/ /S/gcc/testsuite/gcc.target/aarch64/sve/mask_struct_load_3.c -fno-diagnostics-show-caret -fdiagnostics-color=never -march=armv8.2-a+sve -O2 -ftree-vectorize -ffast-math -ffat-lto-objects -S -o mask_struct_load_3.s during RTL pass: reload /S/gcc/testsuite/gcc.target/aarch64/sve/mask_struct_load_3.c: In function 'test_i8_i64_i8_4': /S/gcc/testsuite/gcc.target/aarch64/sve/mask_struct_load_3.c:38:1: internal compiler error: Max. number of generated reload insns per insn is achieved (90) 0xade757 lra_constraints(bool) /S/gcc/lra-constraints.c:4791 0xac68a7 lra(_IO_FILE*) /S/gcc/lra.c:2410 0xa7925f do_reload /S/gcc/ira.c:5462 0xa7925f execute /S/gcc/ira.c:5646 Please submit a full bug report,
[Bug target/83845] [8 regression] new failures after r256620
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83845 rsandifo at gcc dot gnu.org changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|unassigned at gcc dot gnu.org |rsandifo at gcc dot gnu.org --- Comment #2 from rsandifo at gcc dot gnu.org --- Looking at other PRs first, but intend to get to this soon.
[Bug target/83845] [8 regression] new failures after r256620
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83845 --- Comment #3 from rsandifo at gcc dot gnu.org --- FWIW, I now have patches that fix all the big-endian SVE failures. Hope to post them later this week.
[Bug target/83845] [8 regression] new failures after r256620
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83845 rsandifo at gcc dot gnu.org changed: What|Removed |Added URL||https://gcc.gnu.org/ml/gcc- ||patches/2018-01/msg02169.ht ||ml --- Comment #4 from rsandifo at gcc dot gnu.org --- Patched for big-endian failures posted here: https://gcc.gnu.org/ml/gcc-patches/2018-01/msg02169.html I've also posted patches for other big-endian failures. Together they give clean results for aarch64-sve.exp and also seem to give clean results overall when configured with --with-arch=armv8.2-a+sve. The ILP32 failures are being tracked in PR83846.
[Bug target/83845] [8 regression] new failures after r256620
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83845 --- Comment #5 from rsandifo at gcc dot gnu.org --- Author: rsandifo Date: Thu Feb 1 11:02:52 2018 New Revision: 257285 URL: https://gcc.gnu.org/viewcvs?rev=257285&root=gcc&view=rev Log: [AArch64] Tighten aarch64_secondary_reload condition (PR 83845) aarch64_secondary_reload enforced a secondary reload via aarch64_sve_reload_be for memory and pseudo registers, but failed to do the same for subregs of pseudo registers. To avoid this and any similar problems, the patch instead tests for things that the move patterns handle directly; if the operand isn't one of those, we should use the reload pattern instead. The patch fixes an ICE in sve/mask_struct_store_3.c for aarch64_be, where the bogus target description was (rightly) causing LRA to cycle. 2018-02-01 Richard Sandiford gcc/ PR tearget/83845 * config/aarch64/aarch64.c (aarch64_secondary_reload): Tighten check for operands that need to go through aarch64_sve_reload_be. Reviewed-by: James Greenhalgh Modified: trunk/gcc/ChangeLog trunk/gcc/config/aarch64/aarch64.c
[Bug target/83845] [8 regression] new failures after r256620
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83845 Christophe Lyon changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |FIXED --- Comment #6 from Christophe Lyon --- The ICE on aarch64_be has been fixed at r257285.