From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 16398385415B; Tue, 2 May 2023 12:03:50 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 16398385415B DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1683029030; bh=gcQWuM4l4nV4fB7baG2ic/Okmv/E5r1LyxWQsQOktlc=; h=From:To:Subject:Date:In-Reply-To:References:From; b=fEZ/izqARdUT2fZ2cqGiku4qFNej9R8ZJvv9ZLDu6d5HRdVoSjeB7mUCVJSi+iaI1 I7ZMaGdR1rztZsJUIfsrpvfP20mlunh1JMFNX4GJyeRp4l4DvEl09/AoEOHysPWTcz OykVnUJW497LP/C7hrrse8phY3oUdrd4nZooDqT8= From: "cvs-commit at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug tree-optimization/108625] [11 Regression] forwprop introduces new UB Date: Tue, 02 May 2023 12:03:49 +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: 13.0 X-Bugzilla-Keywords: wrong-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: P2 X-Bugzilla-Assigned-To: rguenth at gcc dot gnu.org X-Bugzilla-Target-Milestone: 11.4 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=3D108625 --- Comment #5 from CVS Commits --- The releases/gcc-11 branch has been updated by Richard Biener : https://gcc.gnu.org/g:a0a1905b26751acea523a3bf1bca68d4ddc55c09 commit r11-10675-ga0a1905b26751acea523a3bf1bca68d4ddc55c09 Author: Richard Biener Date: Thu Feb 2 11:09:26 2023 +0100 middle-end/108625 - wrong folding due to misinterpreted ! The following fixes a problem with ! handling in genmatch which isn't conservative enough when intermediate simplifications push to the sequence but the final operation appears to just pick an existing (but in this case newly defined in the sequence) operand. The easiest fix is to disallow adding to the sequence when processing !. PR middle-end/108625 * genmatch.c (expr::gen_transform): Also disallow resimplificat= ion from pushing to lseq with force_leaf. (dt_simplify::gen_1): Likewise. * gcc.dg/pr108625.c: New testcase. (cherry picked from commit 605d1297b91c2c7c23ccfe669e66dda5791d1f55)=