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 C7DE8385DC04; Fri, 17 Apr 2020 12:16:04 +0000 (GMT)
DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C7DE8385DC04
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org;
 s=default; t=1587125764;
 bh=3gkjFP5ERpvDbraYuWs9tM6qfQVnQGwmKRUYZmbhbwQ=;
 h=From:To:Subject:Date:In-Reply-To:References:From;
 b=WX77M4yFmcMFpEIDdLJy1F4zAtmInxMlsQqgy41ChBXMitB8n2cHhC0dQ8xMEv14I
 kRM0JKHrJhK+vBya/Q248j2tKnQ4E4dtc9hcxlThktXBC4OYsCgL8hsrxQzKvVfmlj
 GHnU7PiYvEV2PFIToZRQGomuJdzW6ZKcFsA4lgJU=
From: "rguenther at suse dot de" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug ipa/93385] [10 Regression] wrong code with u128 modulo at -O2
 -fno-dce -fno-ipa-cp -fno-tree-dce
Date: Fri, 17 Apr 2020 12:16:04 +0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: ipa
X-Bugzilla-Version: 10.0
X-Bugzilla-Keywords: diagnostic, wrong-code
X-Bugzilla-Severity: normal
X-Bugzilla-Who: rguenther at suse dot de
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Resolution: 
X-Bugzilla-Priority: P1
X-Bugzilla-Assigned-To: jamborm at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 10.0
X-Bugzilla-Flags: 
X-Bugzilla-Changed-Fields: 
Message-ID: <bug-93385-4-Nhw96W0MZX@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-93385-4@http.gcc.gnu.org/bugzilla/>
References: <bug-93385-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
X-BeenThere: gcc-bugs@gcc.gnu.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Gcc-bugs mailing list <gcc-bugs.gcc.gnu.org>
List-Unsubscribe: <http://gcc.gnu.org/mailman/options/gcc-bugs>,
 <mailto:gcc-bugs-request@gcc.gnu.org?subject=unsubscribe>
List-Archive: <https://gcc.gnu.org/pipermail/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-request@gcc.gnu.org?subject=help>
List-Subscribe: <http://gcc.gnu.org/mailman/listinfo/gcc-bugs>,
 <mailto:gcc-bugs-request@gcc.gnu.org?subject=subscribe>
X-List-Received-Date: Fri, 17 Apr 2020 12:16:04 -0000

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D93385

--- Comment #21 from rguenther at suse dot de <rguenther at suse dot de> ---
On Fri, 17 Apr 2020, jakub at gcc dot gnu.org wrote:

> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D93385
>=20
> --- Comment #20 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
> Looking at tree-ssa-dce.c, it uses remove_phi_node rather than gsi_remove=
 for
> PHIs.  And for non-PHIs, it calls release_defs after gsi_remove.

Btw, I'd much prefer to not first copy the stmts and then remove them.
Instead the DCE "analysis" can be done on the original IL and stmts
be "marked" to be elided during copying.  That saves generating
SSA names and gimple stmts rather than needing to remove them after the
fact.=