diff mbox

[i386,AVX-512] Fix PR target/70662.

Message ID 20160415152153.GC29751@msticlxl57.ims.intel.com
State New
Headers show

Commit Message

Kirill Yukhin April 15, 2016, 3:21 p.m. UTC 
Hi,
On 15 Apr 06:43, H.J. Lu wrote:
> [hjl@gnu-6 gcc]$ /export/build/gnu/gcc-x32/build-x86_64-linux/gcc/xgcc
> -B/export/build/gnu/gcc-x32/build-x86_64-linux/gcc/  -mx32 -mtune=slm
>  -fno-diagnostics-show-caret -fdiagnostics-color=never   -Og
> -fschedule-insns -fno-tree-fre -mavx512vbmi
> --param=max-sched-ready-insns=1 -masm=intel -c -o pr70662.o
> /export/gnu/import/git/sources/gcc/gcc/testsuite/gcc.target/i386/pr70662.c
> /tmp/ccatFXMU.s: Assembler messages:
> /tmp/ccatFXMU.s:240: Error: operand size mismatch for `vpbroadcastd'
> [hjl@gnu-6 gcc]$
> 
>         vpbroadcastd    zmm17{k1}, QWORD PTR [esp+184]
>                                                   ^^^^^^^^ Should be DWORD.
Thanks for catching.

GCC v6 forked 10 mins ago.

Patch in the bottom checked into gcc-5 branch and trunk (v7).

Is it ok to check in to gcc-6 branch?

> 
> H.J.

--
Thanks, K

Comments

Jakub Jelinek April 15, 2016, 3:25 p.m. UTC | #1 
On Fri, Apr 15, 2016 at 06:21:56PM +0300, Kirill Yukhin wrote:
> Hi,
> On 15 Apr 06:43, H.J. Lu wrote:
> > [hjl@gnu-6 gcc]$ /export/build/gnu/gcc-x32/build-x86_64-linux/gcc/xgcc
> > -B/export/build/gnu/gcc-x32/build-x86_64-linux/gcc/  -mx32 -mtune=slm
> >  -fno-diagnostics-show-caret -fdiagnostics-color=never   -Og
> > -fschedule-insns -fno-tree-fre -mavx512vbmi
> > --param=max-sched-ready-insns=1 -masm=intel -c -o pr70662.o
> > /export/gnu/import/git/sources/gcc/gcc/testsuite/gcc.target/i386/pr70662.c
> > /tmp/ccatFXMU.s: Assembler messages:
> > /tmp/ccatFXMU.s:240: Error: operand size mismatch for `vpbroadcastd'
> > [hjl@gnu-6 gcc]$
> > 
> >         vpbroadcastd    zmm17{k1}, QWORD PTR [esp+184]
> >                                                   ^^^^^^^^ Should be DWORD.
> Thanks for catching.
> 
> GCC v6 forked 10 mins ago.
> 
> Patch in the bottom checked into gcc-5 branch and trunk (v7).
> 
> Is it ok to check in to gcc-6 branch?

Ok.

	Jakub
diff mbox

Patch

diff --git a/gcc/config/i386/sse.md b/gcc/config/i386/sse.md
index 4d2927e..1ffb3b9 100644
--- a/gcc/config/i386/sse.md
+++ b/gcc/config/i386/sse.md
@@ -17264,7 +17264,7 @@ 
   if (<MODE>mode == V2DFmode)
     return "vpbroadcastq\t{%1, %0<mask_operand2>|%0<mask_operand2>, %q1}";
 
-  if (GET_MODE_SIZE (GET_MODE_INNER (<MODE>mode)) == 32)
+  if (GET_MODE_SIZE (GET_MODE_INNER (<MODE>mode)) == 4)
     return "v<sseintprefix>broadcast<bcstscalarsuff>\t{%1, %0<mask_operand2>|%0<mask_operand2>, %k1}";
   else
     return "v<sseintprefix>broadcast<bcstscalarsuff>\t{%1, %0<mask_operand2>|%0<mask_operand2>, %q1}";