From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-bugzilla@gcc.gnu.org>
Received: by sourceware.org (Postfix, from userid 48)
	id 5DFC838582A1; Mon, 28 Nov 2022 01:03:15 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 5DFC838582A1
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1669597395;
	bh=q1IE7v53L7nP5ZGlBcLxmdVAXXzxy4V1XCAX5HwYsQk=;
	h=From:To:Subject:Date:In-Reply-To:References:From;
	b=E8E4FtAkaH2d8Pt7kO/w1cvTTTkiksk+AmcTX3UZtK0HkJyWoIBsQX9AMp25/u127
	 wtF/AK5sQpdWy5z3vAEQJzRIdO0HRbnVTIVtDOt2hJQ6LVKF26L3r98fZrwElErCDB
	 3BFH/4jdWh2LgIZXAaurX7/SP3lIegrX8aNGFo40=
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/107748] [13 Regression] Isn't _mm_cvtsbh_ss incorrect?
Date: Mon, 28 Nov 2022 01:03:11 +0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: target
X-Bugzilla-Version: 13.0
X-Bugzilla-Keywords: 
X-Bugzilla-Severity: normal
X-Bugzilla-Who: cvs-commit at gcc dot gnu.org
X-Bugzilla-Status: UNCONFIRMED
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P1
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 13.0
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-107748-4-9KEA4qUIPz@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-107748-4@http.gcc.gnu.org/bugzilla/>
References: <bug-107748-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
List-Id: <gcc-bugs.sourceware.org>

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

--- Comment #10 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:a1ecc5600464f6a62faab246d522b6328badda90

commit r13-4314-ga1ecc5600464f6a62faab246d522b6328badda90
Author: liuhongt <hongtao.liu@intel.com>
Date:   Wed Nov 23 21:58:09 2022 +0800

    Fix incorrect _mm_cvtsbh_ss.

    After supporting real __bf16, the implementation of _mm_cvtsbh_ss went
    wrong.

    The patch add a builtin to generate pslld for the intrinsic, also
    extendbfsf2 is supported with pslld when !HONOR_NANS (BFmode).

    truncsfbf2 is supported with vcvtneps2bf16 when
    !HONOR_NANS (BFmode) && flag_unsafe_math_optimizations.

    gcc/ChangeLog:

            PR target/107748
            * config/i386/avx512bf16intrin.h (_mm_cvtsbh_ss): Refined.
            * config/i386/i386-builtin-types.def (FLOAT_FTYPE_BFLOAT16):
            New function type.
            * config/i386/i386-builtin.def (BDESC): New builtin.
            * config/i386/i386-expand.cc (ix86_expand_args_builtin):
            Handle the builtin.
            * config/i386/i386.md (extendbfsf2): New expander.
            (extendbfsf2_1): New define_insn.
            (truncsfbf2): Ditto.

    gcc/testsuite/ChangeLog:

            * gcc.target/i386/avx512bf16-cvtsbh2ss-1.c: Scan pslld.
            * gcc.target/i386/extendbfsf.c: New test.=