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 939713938803; Wed,  7 Apr 2021 14:22:12 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 939713938803
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/97513] [11 regression] aarch64 SVE regressions since
 r11-3822
Date: Wed, 07 Apr 2021 14:22:12 +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: 11.0
X-Bugzilla-Keywords: 
X-Bugzilla-Severity: normal
X-Bugzilla-Who: cvs-commit at gcc dot gnu.org
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P1
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 11.0
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-97513-4-PdmD8Ap16c@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-97513-4@http.gcc.gnu.org/bugzilla/>
References: <bug-97513-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
X-BeenThere: gcc-bugs@gcc.gnu.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Gcc-bugs mailing list <gcc-bugs.gcc.gnu.org>
List-Unsubscribe: <https://gcc.gnu.org/mailman/options/gcc-bugs>,
 <mailto:gcc-bugs-request@gcc.gnu.org?subject=unsubscribe>
List-Archive: <https://gcc.gnu.org/pipermail/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-request@gcc.gnu.org?subject=help>
List-Subscribe: <https://gcc.gnu.org/mailman/listinfo/gcc-bugs>,
 <mailto:gcc-bugs-request@gcc.gnu.org?subject=subscribe>
X-List-Received-Date: Wed, 07 Apr 2021 14:22:12 -0000

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

--- Comment #8 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Richard Sandiford <rsandifo@gcc.gnu.o=
rg>:

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

commit r11-8030-g2f3d9104610cb2058cf091707a20c1c6eff8d470
Author: Richard Sandiford <richard.sandiford@arm.com>
Date:   Wed Apr 7 15:21:56 2021 +0100

    vect: Restore variable-length SLP permutes [PR97513]

    Many of the gcc.target/sve/slp-perm*.c tests started failing
    after the introduction of separate SLP permute nodes.
    This patch adds variable-length support using a similar
    technique to vect_transform_slp_perm_load.

    As there, the idea is to detect when every permute mask vector
    is the same and can be generated using a regular stepped sequence.
    We can easily handle those cases for variable-length, but still
    need to restrict the general case to constant-length.

    Again copying vect_transform_slp_perm_load, the idea is to distinguish
    the two cases regardless of whether the length is variable or not,
    partly to increase testing coverage and partly because it avoids
    generating redundant trees.

    Doing this means that we can also use SLP for the two-vector
    permute in pr88834.c, which we couldn't before VEC_PERM_EXPR
    nodes were introduced.  The patch therefore makes pr88834.c
    check that we don't regress back to not using SLP and adds
    pr88834_ld3.c to check for the original problem in the PR.

    gcc/
            PR tree-optimization/97513
            * tree-vect-slp.c (vect_add_slp_permutation): New function,
            split out from...
            (vectorizable_slp_permutation): ...here.  Detect cases in which
            all VEC_PERM_EXPRs are guaranteed to have the same stepped
            permute vector and only generate one permute vector for that ca=
se.
            Extend that case to handle variable-length vectors.

    gcc/testsuite/
            * gcc.target/aarch64/sve/pr88834.c: Expect the vectorizer to use
SLP.
            * gcc.target/aarch64/sve/pr88834_ld3.c: New test.=