[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #21 from richard at nildram dot co dot uk 2007-01-06 08:42 --- Subject: Re: [4.0/4.1 regression] internal consistency failure ebotcazou at gcc dot gnu dot org [EMAIL PROTECTED] writes: Eric asked me to weigh in here. My only concern about a backport is Comment #9, which suggests that the patch didn't work. I'm assuming that since we're all talking about backporting it, that comment was mistaken? If indeed there are no known problems, I think it's reasonable to put this on 4.1. Thanks Mark. It seems that a whole new class of internal consistency failures due to the mishandling of REG_NOTEs in the combiner appeared in 4.1.x (for example rtl-optimization/28243 and the blocked rtl-optimization/29329) and up. Richard's patch is the most generic fix available as of today, so backporting it or not to the 4.1.x branch has cascading effects on the other PRs. Maybe the problem reported in comment #9 is another variant. Richard, are you still ready to backport it? If so, I can revert my patch for rtl-optimization/28243 right now to make things easier. Yup, I'm still ready to backport it if the consensus is that the patch is OK for 4.1. I see from later comments that you might be backporting it yourself, but the offer still stands if you haven't started yet. Richard -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #22 from ebotcazou at gcc dot gnu dot org 2007-01-06 08:53 --- Yup, I'm still ready to backport it if the consensus is that the patch is OK for 4.1. I see from later comments that you might be backporting it yourself, but the offer still stands if you haven't started yet. I only resynced my 4.1 branch (it changed a lot recently) and got a baseline, so I've not really started. OK, please do it and I'll only do the reversion of my patch on all branches. Thanks again. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #23 from rsandifo at gcc dot gnu dot org 2007-01-06 14:27 --- Subject: Bug 25514 Author: rsandifo Date: Sat Jan 6 14:26:58 2007 New Revision: 120526 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=120526 Log: gcc/ Backport: 2006-05-23 Richard Sandiford [EMAIL PROTECTED] PR rtl-optimization/27736 * combine.c (replaced_rhs_value): New variable. (combine_instructions): Set it. (distribute_notes): When distributing a note in replaced_rhs_insn, check whether the value was used in replaced_rhs_value. 2006-05-22 Richard Sandiford [EMAIL PROTECTED] PR rtl-optimization/25514 * combine.c (replaced_rhs_insn): New variable. (combine_instructions): Set replaced_rhs_insn when trying to replace a SET_SRC with a REG_EQUAL note. (distribute_notes): Use replaced_rhs_insn when determining the live range of a REG_DEAD register. gcc/testsuite/ 2006-05-22 Richard Sandiford [EMAIL PROTECTED] PR rtl-optimization/25514 * gcc.c-torture/compile/pr25514.c: New test. Added: branches/gcc-4_1-branch/gcc/testsuite/gcc.c-torture/compile/pr25514.c Modified: branches/gcc-4_1-branch/gcc/ChangeLog branches/gcc-4_1-branch/gcc/combine.c branches/gcc-4_1-branch/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #15 from mmitchel at gcc dot gnu dot org 2007-01-05 17:39 --- Eric asked me to weigh in here. My only concern about a backport is Comment #9, which suggests that the patch didn't work. I'm assuming that since we're all talking about backporting it, that comment was mistaken? If indeed there are no known problems, I think it's reasonable to put this on 4.1. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #16 from ebotcazou at gcc dot gnu dot org 2007-01-05 18:10 --- Eric asked me to weigh in here. My only concern about a backport is Comment #9, which suggests that the patch didn't work. I'm assuming that since we're all talking about backporting it, that comment was mistaken? If indeed there are no known problems, I think it's reasonable to put this on 4.1. Thanks Mark. It seems that a whole new class of internal consistency failures due to the mishandling of REG_NOTEs in the combiner appeared in 4.1.x (for example rtl-optimization/28243 and the blocked rtl-optimization/29329) and up. Richard's patch is the most generic fix available as of today, so backporting it or not to the 4.1.x branch has cascading effects on the other PRs. Maybe the problem reported in comment #9 is another variant. Richard, are you still ready to backport it? If so, I can revert my patch for rtl-optimization/28243 right now to make things easier. -- ebotcazou at gcc dot gnu dot org changed: What|Removed |Added CC||mark at codesourcery dot com http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #17 from mark at codesourcery dot com 2007-01-05 18:13 --- Subject: Re: [4.0/4.1 regression] internal consistency failure ebotcazou at gcc dot gnu dot org wrote: Thanks Mark. It seems that a whole new class of internal consistency failures due to the mishandling of REG_NOTEs in the combiner appeared in 4.1.x (for example rtl-optimization/28243 and the blocked rtl-optimization/29329) and up. Richard's patch is the most generic fix available as of today, so backporting it or not to the 4.1.x branch has cascading effects on the other PRs. Maybe the problem reported in comment #9 is another variant. Do you mean positive cascading effects, in that bugs go away, or negative, in that new bugs show up? Do you think that backporting this patch is going to fix some bugs at the expense of breaking others? Thanks, -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #18 from ebotcazou at gcc dot gnu dot org 2007-01-05 18:33 --- Do you mean positive cascading effects, in that bugs go away, or negative, in that new bugs show up? Positive cascading effects if the set of patch is backported to the branch. Do you think that backporting this patch is going to fix some bugs at the expense of breaking others? That will very likely fix again 28243 and help for 29329. I can make a quick pass on the various related PRs for mainline/4.2/4.1 over the week-end and post kind of an overview of the situation before you definitely decide. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #19 from mark at codesourcery dot com 2007-01-05 18:37 --- Subject: Re: [4.0/4.1 regression] internal consistency failure ebotcazou at gcc dot gnu dot org wrote: --- Comment #18 from ebotcazou at gcc dot gnu dot org 2007-01-05 18:33 --- Do you mean positive cascading effects, in that bugs go away, or negative, in that new bugs show up? Positive cascading effects if the set of patch is backported to the branch. Do you think that backporting this patch is going to fix some bugs at the expense of breaking others? That will very likely fix again 28243 and help for 29329. I can make a quick pass on the various related PRs for mainline/4.2/4.1 over the week-end and post kind of an overview of the situation before you definitely decide. I think I can do best by deferring to you, Roger, and Richard. It sounds like everyone's agreed that Richard's changes are the right thing, in principle. It also sounds like they have had little (no?) negative affect in practice. As long as we don't feel that we're unduly introducing risk (fixing one thing but breaking another), I'm all for putting the patches on the branch. I know Richard's busy (he's getting ready to submit a big batch of changes), so if you feel like trying the backport you might want to give it a go. Otherwise, I'm sure Richard will get to it in the relatively near future. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #20 from ebotcazou at gcc dot gnu dot org 2007-01-05 19:03 --- I know Richard's busy (he's getting ready to submit a big batch of changes), so if you feel like trying the backport you might want to give it a go. Otherwise, I'm sure Richard will get to it in the relatively near future. OK, I'm going to give it a whirl. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #14 from ebotcazou at gcc dot gnu dot org 2006-12-21 22:27 --- Not sure what to think about the current state of affairs of the 4.1 branch and, consequently, about this PR. Anyone? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #13 from ebotcazou at gcc dot gnu dot org 2006-11-16 13:08 --- Ping. :-) -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #12 from ebotcazou at gcc dot gnu dot org 2006-11-02 08:03 --- I think Roger is OK in principle with a backport, but the questions are (a) whether we should keep your patch on mainline too and, if not, (b) whether we should revert it on the branches too. Roger, let me know if I've misrepresented you there. I don't think my patch should be taken into account to make the decision. However, I can first revert it everywhere to make the backport easier. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0, 4.1 regression] internal consistency failure
--- Comment #10 from ebotcazou at gcc dot gnu dot org 2006-11-02 07:42 --- Richard, I'm on another problem related to REG_NOTEs in the combiner (PR 29329) present on the 4.1 branch so I think it's time to make a definitive decision on this PR. I'd be in favor of backporting all your changes to the 4.1 branch but I'll let you and Roger decide. Thanks in advance. -- ebotcazou at gcc dot gnu dot org changed: What|Removed |Added CC||sayle at gcc dot gnu dot ||org, ebotcazou at gcc dot ||gnu dot org Component|target |rtl-optimization Summary|[4.0, 4.1 regression] [m68k]|[4.0, 4.1 regression] |internal consistency failure|internal consistency failure http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514
[Bug rtl-optimization/25514] [4.0/4.1 regression] internal consistency failure
--- Comment #11 from rsandifo at gcc dot gnu dot org 2006-11-02 07:55 --- Ack that, thanks Eric. Roger and I did discuss this a bit off-list, but I don't think we came to any conclusion. (Tardiness on my part being the dominant problem, sorry.) I think Roger is OK in principle with a backport, but the questions are (a) whether we should keep your patch on mainline too and, if not, (b) whether we should revert it on the branches too. Roger, let me know if I've misrepresented you there. Richard -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25514