From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 48) id 987D338582A4; Fri, 17 Mar 2023 19:51:39 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 987D338582A4 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1679082699; bh=iuRldshQgXjAhMqHmo+/qRaDFR7y5QRoU2vtDnNOicQ=; h=From:To:Subject:Date:In-Reply-To:References:From; b=KxR0+8dHGQOpLg81xcpV4zM3hgpigRe32hs8PV4+XIN0dJ+EQMWGWavP2TOsqazG7 VOTv5jPuGtQFLs1GVI2mER4N/fUCs5Ufj8Q/XhV/uTsrVRYvOJplFRN/ssrCVWTWLd HLzPsyR1MgF0D1EjkEleTA+ME6dQPmLktcb0khEI= From: "bergner at gcc dot gnu.org" To: gcc-bugs@gcc.gnu.org Subject: [Bug rtl-optimization/109052] Unnecessary reload with -mfpmath=both Date: Fri, 17 Mar 2023 19:51:39 +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: 13.0 X-Bugzilla-Keywords: ra X-Bugzilla-Severity: normal X-Bugzilla-Who: bergner 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: cc 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=3D109052 Peter Bergner changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |bergner at gcc dot gnu.org --- Comment #7 from Peter Bergner --- (In reply to Vladimir Makarov from comment #6) > I'd like to watch what the effect of the current patch would be first.=20= =20 > Although I tested the patch on many targets as usually for LRA the patch > might result in some troubles on some targets. But I hope nothing bad wi= ll > happen. I hit a bootstrap issue on powerpc64le-linux and it looks (still confirming) that this is probably the patch that caused it, since we're ICEing in your = new combine_reload_insn() function. I opened PR109179 for the ICE. during RTL pass: reload addkf3-sw.c: In function =E2=80=98__addkf3_sw=E2=80=99: addkf3-sw.c:51:1: internal compiler error: RTL check: expected elt 3 type '= e' or 'u', have '0' (rtx note) in PATTERN, at rtl.h:1509 51 | } | ^ 0x1148c663 rtl_check_failed_type2(rtx_def const*, int, int, int, char const= *, int, char const*) /home/bergner/gcc/gcc-fsf-mainline-chip-ice-base/gcc/rtl.cc:907 0x10a0a207 PATTERN(rtx_def*) /home/bergner/gcc/gcc-fsf-mainline-chip-ice-base/gcc/rtl.h:1509 0x1278c12b insn_extract(rtx_insn*) =20=20=20=20=20=20=20 /home/bergner/gcc/build/gcc-fsf-mainline-chip-ice-base-regtest/gcc/insn-ext= ract.cc:23 0x1118c9e7 lra_set_insn_recog_data(rtx_insn*) /home/bergner/gcc/gcc-fsf-mainline-chip-ice-base/gcc/lra.cc:1059 0x11194f23 lra_get_insn_recog_data(rtx_insn*) /home/bergner/gcc/gcc-fsf-mainline-chip-ice-base/gcc/lra-int.h:493 0x111b8de7 combine_reload_insn =20=20=20=20=20=20=20 /home/bergner/gcc/gcc-fsf-mainline-chip-ice-base/gcc/lra-constraints.cc:5017 0x111b9e4b lra_constraints(bool) =20=20=20=20=20=20=20 /home/bergner/gcc/gcc-fsf-mainline-chip-ice-base/gcc/lra-constraints.cc:5231 0x111942a3 lra(_IO_FILE*) /home/bergner/gcc/gcc-fsf-mainline-chip-ice-base/gcc/lra.cc:2375 0x110fdd77 do_reload /home/bergner/gcc/gcc-fsf-mainline-chip-ice-base/gcc/ira.cc:5963 0x110fe5df execute /home/bergner/gcc/gcc-fsf-mainline-chip-ice-base/gcc/ira.cc:6149=