[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-20 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 Jakub Jelinek changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-19 Thread ian_harvey at bigpond dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 Ian Harvey changed: What|Removed |Added CC||ian_harvey at bigpond dot com --- Comment

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-19 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #14 from Jakub Jelinek --- Author: jakub Date: Thu Jan 19 15:41:15 2017 New Revision: 244636 URL: https://gcc.gnu.org/viewcvs?rev=244636=gcc=rev Log: PR target/79127 * acinclude.m4 (LIBGFOR_CHECK_AVX512F): Ensure the

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-19 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #13 from Rainer Emrich --- The proposed patch indeed fixes the issue on x86_64-w64-mingw32, libgfortran builds again. The results of a complete testsuite run can be found here:

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-18 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #12 from Jakub Jelinek --- On x86_64-linux and i686-linux it passed bootstrap/regtest and still defines HAVE_AVX512F, which is desirable there.

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-18 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #11 from Rainer Emrich --- (In reply to Jakub Jelinek from comment #10) > Note I have no access to any kind of Windows, so can just bootstrap/regtest > it on Linux to make sure it hasn't regressed anything

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-18 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #10 from Jakub Jelinek --- Note I have no access to any kind of Windows, so can just bootstrap/regtest it on Linux to make sure it hasn't regressed anything there.

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-18 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #9 from Jakub Jelinek --- Created attachment 40536 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40536=edit gcc7-pr79127.patch Untested patch that should make sure HAVE_AVX512F is not enabled if it will not work properly.

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-18 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #8 from Rainer Emrich --- (In reply to Jakub Jelinek from comment #6) > Perhaps the HAVE_AVX512F configure test is too simple for the SEH purposes. > I've tried: > typedef double __m512d

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-18 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #7 from Rainer Emrich --- (In reply to Rainer Emrich from comment #5) > (In reply to Thomas Koenig from comment #1) > > Looks like the binutils on mingw is not able to handle AVX. > > > > I have to

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-18 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #6 from Jakub Jelinek --- Perhaps the HAVE_AVX512F configure test is too simple for the SEH purposes. I've tried: typedef double __m512d __attribute__ ((__vector_size__ (64))); typedef double __m256d __attribute__

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-18 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #5 from Rainer Emrich --- (In reply to Thomas Koenig from comment #1) > Looks like the binutils on mingw is not able to handle AVX. > > I have to confess that I don't know a lot about binutil > versions.

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-18 Thread rai...@emrich-ebersheim.de
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #4 from Rainer Emrich --- Created attachment 40534 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40534=edit assembly Here's the assembly. AFAICS the issue is in matmul_i4_avx512f.

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-18 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #3 from Jakub Jelinek --- Can you please perhaps attach the assembly (so that we can find out if it is reported in the avx, avx2 or avx512f routines (or all of them))?

[Bug target/79127] [7 Regression] Error: invalid register for .seh_savexmm in matmul_i4.c

2017-01-18 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79127 --- Comment #2 from Jakub Jelinek --- If this is indeed something that older versions of binutils don't handle and latest binutils do, then we probably need to reduce the assembly into a small testcase (or corresponding C) and include that in