[Bug target/96551] New: [10 Regression] FAIL: gcc.target/i386/vectorize8.c (internal compiler error)

[Bug target/96551] New: [10 Regression] FAIL: gcc.target/i386/vectorize8.c (internal compiler error)

[hjl.tools at gmail dot com]

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

            Bug ID: 96551
           Summary: [10 Regression] FAIL: gcc.target/i386/vectorize8.c
                    (internal compiler error)
           Product: gcc
           Version: 10.2.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
          Assignee: crazylht at gmail dot com
          Reporter: hjl.tools at gmail dot com
  Target Milestone: ---
            Target: x86_64

ommit f098bc87dcae5646d11a351cfb55d0e1124c7f60
Author: liuhongt <hongtao.liu@intel.com>
Date:   Mon Jul 20 10:13:58 2020 +0800

    Using UNSPEC for vector compare to mask register.

    For rtx like (eq:HI (V8SI 90) (V8SI 91)), cse will take it as a
    boolean value and try to do some optimization. But it is not true for
    vector compare, also other places in rtl passes hold the same
    assumption.

caused:

FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]16[\t ] 6
FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]16[\t ] 6
FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]16[\t ] 6
FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]32[\t ] 6
FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]32[\t ] 6
FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]32[\t ] 6
FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]64[\t ] 6
FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]64[\t ] 6
FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]64[\t ] 6
FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]8[\t ] 6
FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]8[\t ] 6
FAIL: gcc.target/i386/pr92865-1.c scan-assembler-times vmovdq[au]8[\t ] 6
FAIL: gcc.target/i386/vectorize8.c (internal compiler error)
FAIL: gcc.target/i386/vectorize8.c (internal compiler error)
FAIL: gcc.target/i386/vectorize8.c (internal compiler error)
FAIL: gcc.target/i386/vectorize8.c (test for excess errors)
FAIL: gcc.target/i386/vectorize8.c (test for excess errors)
FAIL: gcc.target/i386/vectorize8.c (test for excess errors)

with

$ cd {build_dir}/gcc && make check
RUNTESTFLAGS="i386.exp=gcc.target/i386/vectorize8.c --target_board='unix{-m64\
-march=cascadelake,-m32\ -march=cascadelake}'"

[Bug target/96551] [10/11 Regression] FAIL: gcc.target/i386/vectorize8.c (internal compiler error)

[crazylht at gmail dot com]

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

--- Comment #1 from Hongtao.liu <crazylht at gmail dot com> ---
For `vec_unpacku_float_hi_v16si` `vec_unpacku_float_lo_v16si`

---
diff --git a/gcc/config/i386/sse.md b/gcc/config/i386/sse.md
index cf083ca28aa..2e60f596bc1 100644
--- a/gcc/config/i386/sse.md
+++ b/gcc/config/i386/sse.md
@@ -6973,7 +6973,7 @@

   emit_insn (gen_vec_extract_hi_v16si (tmp[3], operands[1]));
   emit_insn (gen_floatv8siv8df2 (tmp[2], tmp[3]));
-  emit_insn (gen_rtx_SET (k, gen_rtx_LT (QImode, tmp[2], tmp[0])));
+  ix86_expand_mask_vec_cmp (k, LT, tmp[2], tmp[0]);
   emit_insn (gen_addv8df3_mask (tmp[2], tmp[2], tmp[1], tmp[2], k));
   emit_move_insn (operands[0], tmp[2]);
   DONE;
@@ -7020,7 +7020,7 @@
   k = gen_reg_rtx (QImode);

   emit_insn (gen_avx512f_cvtdq2pd512_2 (tmp[2], operands[1]));
-  emit_insn (gen_rtx_SET (k, gen_rtx_LT (QImode, tmp[2], tmp[0])));
+  ix86_expand_mask_vec_cmp (k, LT, tmp[2], tmp[0]);
   emit_insn (gen_addv8df3_mask (tmp[2], tmp[2], tmp[1], tmp[2], k));
   emit_move_insn (operands[0], tmp[2]);
   DONE;
---

[Bug target/96551] [10/11 Regression] FAIL: gcc.target/i386/vectorize8.c (internal compiler error)

[marxin at gcc dot gnu.org]

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

Martin Liška <marxin at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |marxin at gcc dot gnu.org

--- Comment #2 from Martin Liška <marxin at gcc dot gnu.org> ---
*** Bug 96578 has been marked as a duplicate of this bug. ***

[Bug target/96551] [10/11 Regression] FAIL: gcc.target/i386/vectorize8.c (internal compiler error)

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |10.3
           Keywords|                            |ice-on-valid-code
           Priority|P3                          |P2

[Bug target/96551] [10/11 Regression] FAIL: gcc.target/i386/vectorize8.c (internal compiler error)

[crazylht at gmail dot com]

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

--- Comment #3 from Hongtao.liu <crazylht at gmail dot com> ---
a patch is posted at
https://gcc.gnu.org/pipermail/gcc-patches/2020-August/552230.html

[Bug target/96551] [10/11 Regression] FAIL: gcc.target/i386/vectorize8.c (internal compiler error)

[cvs-commit at gcc dot gnu.org]

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

--- Comment #4 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:1aa71af09350b9ff4d2fad88a440b682545682ec

commit r11-2947-g1aa71af09350b9ff4d2fad88a440b682545682ec
Author: liuhongt <hongtao.liu@intel.com>
Date:   Tue Aug 11 11:05:40 2020 +0800

    Refine expander vec_unpacku_float_hi_v16si/vec_unpacku_float_lo_v16si

    gcc/
            PR target/96551
            * config/i386/sse.md (vec_unpacku_float_hi_v16si): For vector
            compare to integer mask, don't use gen_rtx_LT, use
            ix86_expand_mask_vec_cmp instead.
            (vec_unpacku_float_hi_v16si): Ditto.

    gcc/testsuite
            * gcc.target/i386/avx512f-pr96551-1.c: New test.
            * gcc.target/i386/avx512f-pr96551-2.c: New test.

[Bug target/96551] [10/11 Regression] FAIL: gcc.target/i386/vectorize8.c (internal compiler error)

[cvs-commit at gcc dot gnu.org]

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

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

https://gcc.gnu.org/g:2cbd82044b21679fa533445f373374576f638da1

commit r10-8692-g2cbd82044b21679fa533445f373374576f638da1
Author: liuhongt <hongtao.liu@intel.com>
Date:   Tue Aug 11 11:05:40 2020 +0800

    Refine expander vec_unpacku_float_hi_v16si/vec_unpacku_float_lo_v16si

    gcc/
            PR target/96551
            * config/i386/sse.md (vec_unpacku_float_hi_v16si): For vector
            compare to integer mask, don't use gen_rtx_LT, use
            ix86_expand_mask_vec_cmp instead.
            (vec_unpacku_float_hi_v16si): Ditto.

    gcc/testsuite
            * gcc.target/i386/avx512f-pr96551-1.c: New test.
            * gcc.target/i386/avx512f-pr96551-2.c: New test.

[Bug target/96551] [10/11 Regression] FAIL: gcc.target/i386/vectorize8.c (internal compiler error)

[crazylht at gmail dot com]

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

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

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

--- Comment #6 from Hongtao.liu <crazylht at gmail dot com> ---
Fixed in GCC11 and backport to GCC10