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 4DAD1385828E; Wed, 20 Sep 2023 10:13:40 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 4DAD1385828E
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1695204820;
	bh=ngdIQoSyP2YA4FF7GEt72+3VeRwWQpW1HE/ePyp88BU=;
	h=From:To:Subject:Date:In-Reply-To:References:From;
	b=ZVi03xivUlBspF9PTAgZVHre/Tlr0T8IkJWB0Ar23+/KUZpJRLsxiwYzZENRoJMA3
	 aE73M1OQKUL17yr1ydNAnkRYbFW3hTUJNLpXlqdKndgslrXCSofCH6Y9jlZff91w63
	 K8b9Xgx82EhBGtUsZcdY9h26ZZkxk/vekmoS2eEE=
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/111411] [14 regression] ICE when building opus-1.4,
 unrecognizable insn with -fstack-protector-strong
Date: Wed, 20 Sep 2023 10:13:40 +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: ice-on-valid-code
X-Bugzilla-Severity: normal
X-Bugzilla-Who: cvs-commit at gcc dot gnu.org
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: rsandifo at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 14.0
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-111411-4-TnOmx6kyEh@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-111411-4@http.gcc.gnu.org/bugzilla/>
References: <bug-111411-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=3D111411

--- Comment #14 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The releases/gcc-12 branch has been updated by Richard Sandiford
<rsandifo@gcc.gnu.org>:

https://gcc.gnu.org/g:74f99f1adc696f446115f36974a3f94f66294a53

commit r12-9886-g74f99f1adc696f446115f36974a3f94f66294a53
Author: Richard Sandiford <richard.sandiford@arm.com>
Date:   Wed Sep 20 11:13:20 2023 +0100

    aarch64: Fix loose ldpstp check [PR111411]

    aarch64_operands_ok_for_ldpstp contained the code:

      /* One of the memory accesses must be a mempair operand.
         If it is not the first one, they need to be swapped by the
         peephole.  */
      if (!aarch64_mem_pair_operand (mem_1, GET_MODE (mem_1))
           && !aarch64_mem_pair_operand (mem_2, GET_MODE (mem_2)))
        return false;

    But the requirement isn't just that one of the accesses must be a
    valid mempair operand.  It's that the lower access must be, since
    that's the access that will be used for the instruction operand.

    gcc/
            PR target/111411
            * config/aarch64/aarch64.cc (aarch64_operands_ok_for_ldpstp):
Require
            the lower memory access to a mem-pair operand.

    gcc/testsuite/
            PR target/111411
            * gcc.dg/rtl/aarch64/pr111411.c: New test.

    (cherry picked from commit 2d38f45bcca62ca0c7afef4b579f82c5c2a01610)=