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 8AF70385E455; Fri,  5 Jan 2024 12:25:54 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 8AF70385E455
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
	s=default; t=1704457554;
	bh=2zOzJPRHnsV6B8r9lgWttCQbm4dQes/4RKtZ7NWQ394=;
	h=From:To:Subject:Date:In-Reply-To:References:From;
	b=Qzs5fddZYK8mqV07tqtz0IOYdRl/FIT757oZLsdj0qd+Apw2YILe+2XbQ6Iyn5/5z
	 NQo6SaIbWkDDUBUXsiSl31BQVjoHQe/puQ+9hC1w6KatDwO7BWcrGbiyNIGpje5OQ+
	 YwdOeRIv8ycq2nFr9doBt2RlcUAWBTSIHx/RYpD4=
From: "cvs-commit at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug target/113217] [14 Regression][aarch64] ICE in
 rtl_verify_bb_insns, at cfgrtl.cc:2796 since r14-6605-gc0911c6b357ba9
Date: Fri, 05 Jan 2024 12:25:52 +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, patch
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: acoplan at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 14.0
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-113217-4-ZwRyRbaBhG@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-113217-4@http.gcc.gnu.org/bugzilla/>
References: <bug-113217-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=3D113217

--- Comment #6 from GCC Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Alex Coplan <acoplan@gcc.gnu.org>:

https://gcc.gnu.org/g:4b67ec7ff5b1aa9b3b70e9b58afc594b890abeb0

commit r14-6947-g4b67ec7ff5b1aa9b3b70e9b58afc594b890abeb0
Author: Alex Coplan <alex.coplan@arm.com>
Date:   Fri Jan 5 12:25:00 2024 +0000

    aarch64: Further fix for throwing insns in ldp/stp pass [PR113217]

    As the PR shows, the fix in
    r14-6916-g057dc349021660c40699fb5c98fd9cac8e168653 was not complete.
    That fix was enough to stop us trying to move throwing accesses above
    nondebug insns, but due to this code in try_fuse_pair:

      // Placement strategy: push loads down and pull stores up, this should
      // help register pressure by reducing live ranges.
      if (load_p)
        range.first =3D range.last;
      else
        range.last =3D range.first;

    we would still try to move stores up above any debug insns that occurred
    immediately after the previous nondebug insn.  This patch fixes that by
    narrowing the move range in the case that the second access is throwing
    to exactly the range of that insn.

    Note that we still need the fix to latest_hazard_before mentioned above
    so as to ensure we select a suitable base and reject pairs if it isn't
    viable to form the pair at the end of the BB.

    gcc/ChangeLog:

            PR target/113217
            * config/aarch64/aarch64-ldp-fusion.cc
            (ldp_bb_info::try_fuse_pair): If the second access can throw,
            narrow the move range to exactly that insn.

    gcc/testsuite/ChangeLog:

            PR target/113217
            * g++.dg/pr113217.C: New test.=