[Bug target/96667] New: FAIL: gcc.target/i386/avx512bw-pr96246-1.c

[Bug target/96667] New: FAIL: gcc.target/i386/avx512bw-pr96246-1.c

[crazylht at gmail dot com]

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

            Bug ID: 96667
           Summary: FAIL: gcc.target/i386/avx512bw-pr96246-1.c
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: unassigned at gcc dot gnu.org
          Reporter: crazylht at gmail dot com
  Target Milestone: ---
            Target: i386, x86-64

On Linux/x86_64,

7123217afb33d4a2860f552ad778a819cc8dea5e is the first bad commit
commit 7123217afb33d4a2860f552ad778a819cc8dea5e
Author: liuhongt <hongtao.liu@intel.com>
Date:   Tue Jul 21 15:25:20 2020 +0800

    Merge two define_insn: <avx512>_blendm<mode>, <avx512>_load<mode>_mask.

caused

FAIL: gcc.target/i386/avx512bw-pr96246-1.c

with GCC configured with

../../gcc/configure
--prefix=/local/skpandey/gccwork/toolwork/gcc-bisect-master/master/r11-2683/usr
--enable-clocale=gnu --with-system-zlib --with-demangler-in-ld
--with-fpmath=sse --enable-languages=c,c++,fortran --enable-cet --without-isl
--enable-libmpx x86_64-linux --disable-bootstrap

To reproduce:

$ cd {build_dir}/gcc && make check
RUNTESTFLAGS="i386.exp=gcc.target/i386/avx512bw-pr96246-1.c
--target_board='unix{-m64,-m32}'"

[Bug target/96667] FAIL: gcc.target/i386/avx512bw-pr96246-1.c

[crazylht at gmail dot com]

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

--- Comment #1 from Hongtao.liu <crazylht at gmail dot com> ---
Testcase need to be adjusted.
I'll rewrite testcase with cpp source file, then vector compare operator could
be used directly.

--- a/gcc/testsuite/gcc.target/i386/avx512bw-pr96246-1.c
+++ b/gcc/testsuite/g++.target/i386/avx512bw-pr96246-1.C
@@ -1,8 +1,8 @@
 /* PR target/96246 */
 /* { dg-do compile } */
-/* { dg-options "-O2 -ftree-vectorize -mavx512bw" } */
-/* { dg-final { scan-assembler-times "vpblendm\[bwdq\]\[\t ]" 4 } } */
-/* { dg-final { scan-assembler-times "vblendmp\[sd\]\[\t ]" 2 } } */
+/* { dg-options "-O2 -std=c++14 -mavx512bw" } */
+/* { dg-final { scan-assembler-times "vpblendm\[bwdq\]\[\t \]" 4 } } */
+/* { dg-final { scan-assembler-times "vblendmp\[sd\]\[\t \]" 2 } } */

 typedef char v64qi __attribute__((vector_size (64)));
 typedef short v32hi __attribute__((vector_size (64)));
@@ -16,10 +16,7 @@ typedef double v8df __attribute__((vector_size (64)));
   __attribute__ ((noipa))                              \
   foo_##vtype (vtype a, vtype b, vtype c, vtype d)     \
   {                                                    \
-    vtype e;                                           \
-    for (int i = 0; i != num; i++)                     \
-      e[i] = a[i] > b[i] ? c[i] : d[i];                        \
-    return e;                                          \
+    return a > b ? c : d;                              \
   }

 COMPILE_TEST (v64qi, 64);

this would avoid impact of vectorizer.

[Bug target/96667] FAIL: gcc.target/i386/avx512bw-pr96246-1.c

[cvs-commit at gcc dot gnu.org]

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

--- Comment #2 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by hongtao Liu <liuhongt@gcc.gnu.org>:

https://gcc.gnu.org/g:55290635d10ea3d18b0a45aa2e6f59405b8c1f54

commit r11-2856-g55290635d10ea3d18b0a45aa2e6f59405b8c1f54
Author: liuhongt <hongtao.liu@intel.com>
Date:   Tue Aug 18 15:06:01 2020 +0800

    Adjust testcase.

    Rewriting testcase with cpp source file, then compare operator could
    be used directly for vector, this would avoid impact of vectorizer.

    gcc/testsuite/ChangeLog:
            PR target/96667
            * gcc.target/i386/avx512bw-pr96246-1.c: Moved to...
            * g++.target/i386/avx512bw-pr96246-1.C: ...here.
            * gcc.target/i386/avx512bw-pr96246-2.c: Moved to...
            * g++.target/i386/avx512bw-pr96246-2.C: ...here.
            * gcc.target/i386/avx512vl-pr96246-1.c: Moved to...
            * g++.target/i386/avx512vl-pr96246-1.C: ...here.
            * gcc.target/i386/avx512vl-pr96246-2.c: Moved to...
            * g++.target/i386/avx512vl-pr96246-2.C: ...here.

[Bug target/96667] FAIL: gcc.target/i386/avx512bw-pr96246-1.c

[crazylht at gmail dot com]

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

Hongtao.liu <crazylht at gmail dot com> changed:

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

--- Comment #3 from Hongtao.liu <crazylht at gmail dot com> ---
Fixed in GCC11.