https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
Jakub Jelinek changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #21 from Jakub Jelinek ---
Author: jakub
Date: Mon Jun 25 17:55:15 2018
New Revision: 262103
URL: https://gcc.gnu.org/viewcvs?rev=262103&root=gcc&view=rev
Log:
PR target/84786
* config/i386/sse.md (vshift_count): New
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #20 from Jakub Jelinek ---
Author: jakub
Date: Mon Jun 25 12:58:03 2018
New Revision: 262016
URL: https://gcc.gnu.org/viewcvs?rev=262016&root=gcc&view=rev
Log:
2018-06-25 Jakub Jelinek
PR target/84786
* gcc.target
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #19 from Jakub Jelinek ---
Author: jakub
Date: Mon Jun 25 12:57:04 2018
New Revision: 262015
URL: https://gcc.gnu.org/viewcvs?rev=262015&root=gcc&view=rev
Log:
PR target/84786
* config/i386/sse.md (vshift_count): New
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #18 from Jakub Jelinek ---
Author: jakub
Date: Mon Jun 25 12:48:29 2018
New Revision: 262014
URL: https://gcc.gnu.org/viewcvs?rev=262014&root=gcc&view=rev
Log:
PR target/84786
* config/i386/sse.md (vshift_count): New
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #17 from Jakub Jelinek ---
Author: jakub
Date: Fri Jun 22 20:36:31 2018
New Revision: 261919
URL: https://gcc.gnu.org/viewcvs?rev=261919&root=gcc&view=rev
Log:
Backported from mainline
2018-03-13 Jakub Jelinek
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #16 from Jakub Jelinek ---
Created attachment 44312
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=44312&action=edit
gcc7-pr84786.patch
Patch I'm going to bootstrap/regtest on 7.x branch now.
On the trunk/in 8.x this got fixed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #15 from Matthias Kretz ---
Here's an idea for a test case (https://godbolt.org/g/SjM2HE: it appears fixed
on GCC 8):
typedef unsigned short V __attribute__((vector_size (16)));
V foo (V x, int y)
{
x <<= y;
asm volatile (""::"x
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #14 from Matthias Kretz ---
I applied both patches to my GCC 7.2 installation and as a result my complete
testsuite passes now. Anything else I can help with?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #13 from Matthias Kretz ---
I'll try to apply it locally and will report my findings.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #12 from Jakub Jelinek ---
For 7.x I think we need:
--- gcc/config/i386/sse.md.jj 2018-03-05 17:04:45.820743323 +0100
+++ gcc/config/i386/sse.md 2018-03-26 17:29:00.967880855 +0200
@@ -10687,7 +10687,7 @@
[(set (match_operan
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #11 from Matthias Kretz ---
Created attachment 43762
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43762&action=edit
test case that produces incorrect vpsrlw
Compiled with `g++-7 -std=c++17 -O0 -fabi-version=0 -fabi-compat-ver
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #10 from Matthias Kretz ---
This is all I have right now:
TID 0 SDE-ERROR: Executed instruction not valid for specified chip (KNL):
0x70d281: vpsrlw xmm0, xmm0, xmm16
Image:
/home/travis/build/VcDevel/Vc/build-Experimental/c2dd920conc
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #9 from Jakub Jelinek ---
(In reply to Matthias Kretz from comment #8)
> There seems to be a similar bug for vpsrlw and vpsllw. Do you need a
> testcase? (It's hard to hit the bug... just had one occur on a Travis CI
> build)
I don't
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #8 from Matthias Kretz ---
There seems to be a similar bug for vpsrlw and vpsllw. Do you need a testcase?
(It's hard to hit the bug... just had one occur on a Travis CI build)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #7 from Jakub Jelinek ---
Fixed for 8.1+ so far.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #6 from Jakub Jelinek ---
Author: jakub
Date: Tue Mar 13 08:03:28 2018
New Revision: 258475
URL: https://gcc.gnu.org/viewcvs?rev=258475&root=gcc&view=rev
Log:
PR target/84786
* config/i386/sse.md (sse2_loadhpd): Use Y
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #5 from Jakub Jelinek ---
Created attachment 43628
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43628&action=edit
gcc8-pr84786.patch
Untested fix.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #4 from Jakub Jelinek ---
Simplified testcase for the testsuite:
/* PR target/84786 */
/* { dg-do run } */
/* { dg-options "-mavx512f -mno-avx512vl -O2" } */
/* { dg-require-effective-target avx512f } */
#include "avx512f-check.h"
t
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
Jakub Jelinek changed:
What|Removed |Added
Keywords||wrong-code
Status|UNCONFIRME
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
--- Comment #2 from Matthias Kretz ---
Created attachment 43618
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43618&action=edit
unreduced testcase
Compile with `g++ -std=c++17 -O2 -march=knl -o knl-fail knl-fail.cpp`.
The function `Tests:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84786
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #1
22 matches
Mail list logo
