From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 0C4503858430; Fri, 29 Oct 2021 09:09:48 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 0C4503858430 From: "rguenth at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug tree-optimization/102880] [12 Regression] Dead Code Elimination Regression at -O3 Date: Fri, 29 Oct 2021 09:09:48 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: tree-optimization X-Bugzilla-Version: 12.0 X-Bugzilla-Keywords: X-Bugzilla-Severity: normal X-Bugzilla-Who: rguenth at gcc dot gnu.org X-Bugzilla-Status: ASSIGNED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: rguenth at gcc dot gnu.org X-Bugzilla-Target-Milestone: 12.0 X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: attachments.created 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 X-BeenThere: gcc-bugs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-bugs mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 29 Oct 2021 09:09:49 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D102880 --- Comment #5 from Richard Biener --- Created attachment 51699 --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=3D51699&action=3Dedit hack to make forwarders This is an overly simple attempt to make forwarders which works to restore = the DCE. Given this it probably makes sense to at least move the early merge_phi pass after CD-DCE. That doesn't help this testcase since regular CFG cleanup performs the merging in our case. It's remove_forwarder_block will perform this. An alternative band-aid would be to resolve the regression by not running CFG cleanup after DSE (we're failing to run it when we empty a BB as well, something we might want to fix instead). Undoing PHI merging might be also a good thing to perform before going out = of SSA. And merge_phi itself might do this as a 2nd step.=