From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 6763B384AB4C; Tue, 30 Apr 2024 08:26:50 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 6763B384AB4C DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1714465610; bh=3r3dz8sE7KSTPyx/UV75pEqWW/ovLbtXS14QJSIEoJs=; h=From:To:Subject:Date:In-Reply-To:References:From; b=lIF9+SkpiVn1ZDFRb0O4TCHzc/RTIOwLot1prBwDMce4UaLmX+EvoMXWI99VsJALw aLCAlXAMxSbRAPYQQfjxB5rIORGYJ3IrRGmmw19v3KArKoT3HU5xHrUB+3r78xKaVN RYiTrecldEHdnjtquUTVUP2PfCRTmlGUvzxQy00E= From: "cvs-commit at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug target/111600] [14/15 Regression] RISC-V bootstrap time regression Date: Tue, 30 Apr 2024 08:26:43 +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: compile-time-hog, needs-bisection X-Bugzilla-Severity: normal X-Bugzilla-Who: cvs-commit at gcc dot gnu.org X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Resolution: X-Bugzilla-Priority: P4 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: 14.0 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: Message-ID: In-Reply-To: References: 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: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D111600 --- Comment #33 from GCC Commits --- The releases/gcc-13 branch has been updated by Richard Biener : https://gcc.gnu.org/g:440a612dbadfae5887ec7c9b3ab2fca88b065366 commit r13-8662-g440a612dbadfae5887ec7c9b3ab2fca88b065366 Author: Richard Biener Date: Thu Sep 28 11:51:30 2023 +0200 target/111600 - avoid deep recursion in access diagnostics pass_waccess::check_dangling_stores uses recursion to traverse the CFG. The following changes this to use a heap allocated worklist to avoid blowing the stack. Instead of using a better iteration order it tries hard to preserve the current iteration order to avoid new false positives to pop up since the set of stores we keep track isn't properly modeling flow, so what is diagnosed and what not is quite random. We are also lacking the ideal RPO compute on the inverted graph that would just ignore reverse unreachable code (as the current iteration scheme does). PR target/111600 * gimple-ssa-warn-access.cc (pass_waccess::check_dangling_store= s): Use a heap allocated worklist for CFG traversal instead of recursion. (cherry picked from commit f194c684a28a5d449bd034a2c604d04ba465e4fe)=