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 0BB993849AC4; Thu, 16 May 2024 00:41:19 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 0BB993849AC4
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1715820080;
	bh=l7xqZFL1UG5eLpeV7afgOTeLQdm/MY0cB56DJ+5xrqc=;
	h=From:To:Subject:Date:In-Reply-To:References:From;
	b=Zl88ViwAnyK+0BC3SSAUCE4TtPerl6fFeadsNH0asfjZGiH7uza1GdtEPnNCg/FRt
	 abyzB0Y3vFWxL/NzD96uBoPRKLs2/Fu8f+q9ooqz8n8sDEnE5JPS+LP6Wano/HxKjS
	 M30JNKrg5y9Y8w4cm1ibdVPQs7C2RyI6lK33FhJs=
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/114514] v16qi >> 7 can be optimized with vpcmpgtb
Date: Thu, 16 May 2024 00:41:19 +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: 14.0
X-Bugzilla-Keywords: missed-optimization
X-Bugzilla-Severity: enhancement
X-Bugzilla-Who: cvs-commit at gcc dot gnu.org
X-Bugzilla-Status: NEW
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-114514-4-SODYh43C5v@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-114514-4@http.gcc.gnu.org/bugzilla/>
References: <bug-114514-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=3D114514

--- Comment #5 from GCC 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:090714e6cf8029f4ff8883dce687200024adbaeb

commit r15-530-g090714e6cf8029f4ff8883dce687200024adbaeb
Author: liuhongt <hongtao.liu@intel.com>
Date:   Wed May 15 10:56:24 2024 +0800

    Set d.one_operand_p to true when TARGET_SSSE3 in
ix86_expand_vecop_qihi_partial.

    pshufb is available under TARGET_SSSE3, so
    ix86_expand_vec_perm_const_1 must return true when TARGET_SSSE3.

    With the patch under -march=3Dx86-64-v2

    v8qi
    foo (v8qi a)
    {
      return a >> 5;
    }

    <       pmovsxbw        %xmm0, %xmm0
    <       psraw   $5, %xmm0
    <       pshufb  .LC0(%rip), %xmm0

            vs.

    >       movdqa  %xmm0, %xmm1
    >       pcmpeqd %xmm0, %xmm0
    >       pmovsxbw        %xmm1, %xmm1
    >       psrlw   $8, %xmm0
    >       psraw   $5, %xmm1
    >       pand    %xmm1, %xmm0
    >       packuswb        %xmm0, %xmm0

    Although there's a memory load from constant pool, but it should be
    better when it's inside a loop. The load from constant pool can be
    hoist out. it's 1 instruction vs 4 instructions.

    <       pshufb  .LC0(%rip), %xmm0

    vs.

    >       pcmpeqd %xmm0, %xmm0
    >       psrlw   $8, %xmm0
    >       pand    %xmm1, %xmm0
    >       packuswb        %xmm0, %xmm0

    gcc/ChangeLog:

            PR target/114514
            * config/i386/i386-expand.cc (ix86_expand_vecop_qihi_partial):
            Set d.one_operand_p to true when TARGET_SSSE3.

    gcc/testsuite/ChangeLog:

            * gcc.target/i386/pr114514-shufb.c: New test.=