From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id C59863857C5A; Thu, 23 Jul 2020 12:56:49 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org C59863857C5A DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1595509009; bh=TLmWN7OP+wZ5W/fu5s5RsvLmknRPn//TauCh9M6BSHY=; h=From:To:Subject:Date:In-Reply-To:References:From; b=dLPaRotRNRn6COE3qJRFP7bWVTRp7Z3BGM7VaBKsbYxS3io2VrpX39VDeTAwSliIu c6N8LjSlfwl1FNC/x5hXGskNT9a15rlWQ1EAhSYW2ki7CWysMKp17UpUxiQXVSbNss T4SX2cSbkXAy7xMf1d21fX7w15dm8zllSw2dWjVs= From: "segher at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug rtl-optimization/96298] [11 Regression] wrong code with -O -fno-tree-forwprop Date: Thu, 23 Jul 2020 12:56:49 +0000 X-Bugzilla-Reason: CC X-Bugzilla-Type: changed X-Bugzilla-Watch-Reason: None X-Bugzilla-Product: gcc X-Bugzilla-Component: rtl-optimization X-Bugzilla-Version: 11.0 X-Bugzilla-Keywords: needs-bisection, wrong-code X-Bugzilla-Severity: normal X-Bugzilla-Who: segher at gcc dot gnu.org X-Bugzilla-Status: NEW X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: 11.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 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: Thu, 23 Jul 2020 12:56:49 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D96298 --- Comment #5 from Segher Boessenkool --- Trying 17, 14 -> 18: 17: r147:DI=3Dr145:DI^r121:DI REG_DEAD r145:DI REG_DEAD r121:DI 14: r144:DI=3Dr142:DI^r121:DI REG_DEAD r142:DI 18: r148:DI=3Dr144:DI^r147:DI REG_DEAD r147:DI REG_DEAD r144:DI (no other insn here not mentioned uses r144 or r147) This could simplify to just r142^r145. But we go completely wrong here, instead: Failed to match this instruction: (set (reg:DI 148) (xor:DI (xor:DI (reg:DI 145) (reg:DI 142)) (reg:DI 121 [ _12 ]))) Successfully matched this instruction: (set (reg:DI 147) (xor:DI (reg:DI 145) (reg:DI 142))) Successfully matched this instruction: (set (reg:DI 148) (xor:DI (reg:DI 147) (reg:DI 121 [ _12 ]))) allowing combination of insns 14, 17 and 18 original costs 4 + 4 + 4 =3D 12 replacement costs 4 + 4 =3D 8 deferring deletion of insn with uid =3D 14. modifying insn i2 17: r147:DI=3Dr145:DI^r142:DI REG_DEAD r142:DI REG_DEAD r145:DI deferring rescan insn with uid =3D 17. modifying insn i3 18: r148:DI=3Dr147:DI^r121:DI REG_DEAD r121:DI REG_DEAD r147:DI deferring rescan insn with uid =3D 18. That sounds like simplify-rtx yes.=