From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id A1275386EC42; Wed, 16 Sep 2020 13:49:24 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org A1275386EC42 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1600264164; bh=+VJBuHb8P1/umPUsiXf3wQfj4GGNWg3e84mGuK20OKw=; h=From:To:Subject:Date:In-Reply-To:References:From; b=DAtRt7ieYEPaDieUxecGsBkD1uRi+j0GJwXilElit2T1V8iRDDViIjbKHd0N88GH+ 6mYn1K/sLXesJqLIq7tZ6Dc/TeAz8ZoPmPwgykuKhd2WKGrAeULo7l/5lFKKZ44QMF pC/mQVkDnw9uExiElcOWpQJrOlztxyfSeI0dn1xU= From: "rguenth at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug rtl-optimization/97071] Fails to CSE / inherit constant pool load Date: Wed, 16 Sep 2020 13:49:24 +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: missed-optimization X-Bugzilla-Severity: normal X-Bugzilla-Who: rguenth at gcc dot gnu.org X-Bugzilla-Status: UNCONFIRMED X-Bugzilla-Resolution: X-Bugzilla-Priority: P3 X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org X-Bugzilla-Target-Milestone: --- X-Bugzilla-Flags: X-Bugzilla-Changed-Fields: cf_gcctarget keywords 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: Wed, 16 Sep 2020 13:49:24 -0000 https://gcc.gnu.org/bugzilla/show_bug.cgi?id=3D97071 Richard Biener changed: What |Removed |Added ---------------------------------------------------------------------------- Target| |powerpc64le Keywords| |missed-optimization --- Comment #1 from Richard Biener --- Right before combine we see the following, still nicely with REG_EQUAL notes (insn 7 17 9 2 (set (reg:DF 119) (mem/u/c:DF (unspec:DI [ (symbol_ref/u:DI ("*.LC0") [flags 0x82]) (reg:DI 2 2) ] UNSPEC_TOCREL) [0 S8 A64])) "t.c":3:20 533 {*movdf_hardfloat64}=20=20=20 (expr_list:REG_EQUAL (const_double:DF -3.0e+0 [-0x0.cp+2]) (nil))) (insn 9 7 14 2 (set (reg:DF 121) (mem/u/c:DF (unspec:DI [ (symbol_ref/u:DI ("*.LC2") [flags 0x82]) (reg:DI 2 2) ] UNSPEC_TOCREL) [0 S8 A64])) "t.c":3:20 533 {*movdf_hardfloat64}=20=20=20 (expr_list:REG_EQUAL (const_double:DF 3.0e+0 [0x0.cp+2]) (nil))) (insn 14 9 15 2 (set (reg/i:DF 33 1) (fma:DF (reg:DF 124) (reg:DF 119) (reg:DF 121))) "t.c":4:1 894 {*fmadf4_fpr} (expr_list:REG_DEAD (reg:DF 124) (expr_list:REG_DEAD (reg:DF 121) (expr_list:REG_DEAD (reg:DF 119) (nil))))) eventually the easiest pass to teach this to is fwprop though as it already works DF DEF -> USE. Alternatively PRE could make the subtract and/or the negated value anticipated.=