public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/95713] [10/11 Regression] ICE in emit_move_insn when converting int2 vector to short2 vector for -march=skylake-avx512 since r10-5031-g78307657cf9675bc
Date: Thu, 18 Jun 2020 10:18:04 +0000	[thread overview]
Message-ID: <bug-95713-4-8SkvXqGjS8@http.gcc.gnu.org/bugzilla/> (raw)
In-Reply-To: <bug-95713-4@http.gcc.gnu.org/bugzilla/>

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

--- Comment #8 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-10 branch has been updated by Jakub Jelinek
<jakub@gcc.gnu.org>:

https://gcc.gnu.org/g:0c7d18d51a8e9abd085afd0b01743662fd0419fa

commit r10-8323-g0c7d18d51a8e9abd085afd0b01743662fd0419fa
Author: Jakub Jelinek <jakub@redhat.com>
Date:   Thu Jun 18 12:08:06 2020 +0200

    forwprop: simplify_vector_constructor follow-up fix [PR95713]

    As the following testcase shows, the exception for the aarch64
    vec_pack_trunc_di is not sufficient on x86, the halfvectype
    "vectors" have SImode but the x86 vec_pack_trunc_si meant for
    the bool bitmasks combines 2x SImode into DImode, while in the
    testcase the halfvectype is 1x SImode "vector" with SImode and
    result is 2x HImode "vector" with SImode.

    Richard Sandiford's reply:
    FWIW, since the aarch64 case was only found by inspection and might
    not be useful, personally I'd prefer to drop that case after all.

    2020-06-18  Jakub Jelinek  <jakub@redhat.com>

            PR target/95713
            * tree-ssa-forwprop.c (simplify_vector_constructor): Don't allow
            scalar mode halfvectype other than vector boolean for
            VEC_PACK_TRUNC_EXPR.

            * gcc.dg/pr95713.c: New test.

    (cherry picked from commit 9a875db43cd4210843c480c8df26a1d6624d057d)

  parent reply	other threads:[~2020-06-18 10:18 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-16 20:12 [Bug rtl-optimization/95713] New: [10/11 Regression] ICE in emit_move_insn when converting int2 vector to short2 vector for -march=skylake-avx512 -m32 kretz at kde dot org
2020-06-17  6:23 ` [Bug target/95713] [10/11 Regression] ICE in emit_move_insn when converting int2 vector to short2 vector for -march=skylake-avx512 since r10-5031-g78307657cf9675bc marxin at gcc dot gnu.org
2020-06-17  6:54 ` rguenth at gcc dot gnu.org
2020-06-17  6:59 ` rguenth at gcc dot gnu.org
2020-06-17  8:46 ` jakub at gcc dot gnu.org
2020-06-17  9:53 ` jakub at gcc dot gnu.org
2020-06-18  8:25 ` kretz at kde dot org
2020-06-18 10:13 ` cvs-commit at gcc dot gnu.org
2020-06-18 10:18 ` cvs-commit at gcc dot gnu.org [this message]
2020-06-29  9:56 ` jakub at gcc dot gnu.org

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bug-95713-4-8SkvXqGjS8@http.gcc.gnu.org/bugzilla/ \
    --to=gcc-bugzilla@gcc.gnu.org \
    --cc=gcc-bugs@gcc.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).