public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
From: "segher at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org> To: gcc-bugs@gcc.gnu.org Subject: [Bug debug/99830] [11 Regression] ICE: in lra_eliminate_regs_1, at lra-eliminations.c:659 with -O2 -fno-expensive-optimizations -fno-split-wide-types -g Date: Fri, 09 Apr 2021 21:29:51 +0000 [thread overview] Message-ID: <bug-99830-4-3iBHlZb4U6@http.gcc.gnu.org/bugzilla/> (raw) In-Reply-To: <bug-99830-4@http.gcc.gnu.org/bugzilla/> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=99830 --- Comment #14 from Segher Boessenkool <segher at gcc dot gnu.org> --- (In reply to Jakub Jelinek from comment #13) > Seems the exact spot where the clobber is optimized away is e.g. when > simplify_and_const_int_1 (SImode, (ashift:SI (subreg:SI (and:TI (clobber:TI > (const_int 0 [0])) (const_int 255 [0xff])) 0) (const_int 16 [0x10])), 255); > is called. > It calls nonzero_bits, nonzero_bits sees VARYING << 16 and so returns > 0xffff0000, > /* Turn off all bits in the constant that are known to already be zero. > Thus, if the AND isn't needed at all, we will have CONSTOP == > NONZERO_BITS > which is tested below. */ > > constop &= nonzero; > > /* If we don't have any bits left, return zero. */ > if (constop == 0) > return const0_rtx; > > So, are you suggesting that in all such spots we need to test side_effects_p > and punt? Yes, you need to do check side_effects_p *everywhere* you can potentially remove a side effect. This is not specific to combine, even. > Note, simplify_and_const_int_1 already starts with: > if (GET_CODE (varop) == CLOBBER) > return NULL_RTX; > so it would need to use > if (side_effects_p (varop)) > return NULL_RTX; > instead. Yeah. This no longer disallows a VOIDmode clobber, but we should not see those here anyway. You'll need the same change a few lines later, btw: varop = force_to_mode (varop, mode, constop, 0); /* If VAROP is a CLOBBER, we will fail so return it. */ if (GET_CODE (varop) == CLOBBER) return varop; (you only need that second one, even, force_to_mode immediately returns its arg if it is a clobber).
next prev parent reply other threads:[~2021-04-09 21:29 UTC|newest] Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-03-30 14:11 [Bug rtl-optimization/99830] New: " zsojka at seznam dot cz 2021-03-30 16:14 ` [Bug rtl-optimization/99830] " acoplan at gcc dot gnu.org 2021-03-31 7:53 ` rguenth at gcc dot gnu.org 2021-04-08 13:21 ` rguenth at gcc dot gnu.org 2021-04-08 13:27 ` jakub at gcc dot gnu.org 2021-04-08 14:45 ` [Bug debug/99830] " jakub at gcc dot gnu.org 2021-04-09 12:29 ` segher at gcc dot gnu.org 2021-04-09 12:36 ` jakub at gcc dot gnu.org 2021-04-09 12:38 ` segher at gcc dot gnu.org 2021-04-09 13:27 ` jakub at gcc dot gnu.org 2021-04-09 14:23 ` jakub at gcc dot gnu.org 2021-04-09 17:09 ` segher at gcc dot gnu.org 2021-04-09 17:31 ` jakub at gcc dot gnu.org 2021-04-09 18:01 ` segher at gcc dot gnu.org 2021-04-09 19:10 ` jakub at gcc dot gnu.org 2021-04-09 19:10 ` jakub at gcc dot gnu.org 2021-04-09 21:29 ` segher at gcc dot gnu.org [this message] 2021-04-12 16:20 ` jakub at gcc dot gnu.org 2021-04-12 23:01 ` cvs-commit at gcc dot gnu.org 2021-04-12 23:03 ` jakub at gcc dot gnu.org 2021-04-20 9:46 ` cvs-commit at gcc dot gnu.org 2021-04-20 23:34 ` cvs-commit at gcc dot gnu.org 2021-04-22 16:52 ` cvs-commit at gcc dot gnu.org
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=bug-99830-4-3iBHlZb4U6@http.gcc.gnu.org/bugzilla/ \ --to=gcc-bugzilla@gcc.gnu.org \ --cc=gcc-bugs@gcc.gnu.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).