--- Comment #19 from mmitchel at gcc dot gnu dot org 2006-05-25 02:36
---
Will not be fixed in 4.1.1; adjust target milestone to 4.1.2.
--
mmitchel at gcc dot gnu dot org changed:
What|Removed |Added
---
--- Comment #20 from pinskia at gcc dot gnu dot org 2006-07-20 11:13
---
This was fixed, correct?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25335
--- Comment #21 from amylaar at spamcop dot net 2006-07-20 17:08 ---
Subject: Re: [4.1/4.2 Regression] reload leaves
insns from earlier passes around: fatal for postinc
Quoting pinskia at gcc dot gnu dot org <[EMAIL PROTECTED]>:
>
>
> --- Comment #20 from pinskia at gcc do
--- Comment #22 from pinskia at gcc dot gnu dot org 2006-07-20 22:14
---
Fixed.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNE
--- Comment #3 from mmitchel at gcc dot gnu dot org 2005-12-19 19:26
---
Do we have a testcase for this bug on a primary/secondary platform, or is this
something CRIS-specific?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25335
--- Comment #4 from hp at gcc dot gnu dot org 2005-12-19 20:10 ---
Incomplete analysis, can't reply to the second part of comment #3.
"No" to the first part.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25335
--- Comment #5 from mmitchel at gcc dot gnu dot org 2006-01-15 22:11
---
I'm leaving this at P3 until we have more information. Clearly, if this is a
general problem that affets primary/secondary targets, we should increase the
priority.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi
--- Comment #6 from amylaar at gcc dot gnu dot org 2006-01-30 20:24 ---
> Reload seems to need two rounds, but the emitted reload insns for each pass
> is left around. This is exposed but not actually caused by the fix for
> PR middle-end/24912.
Reload should be only called once per fun
--- Comment #7 from hp at gcc dot gnu dot org 2006-01-30 21:53 ---
Re: "needing two rounds". Looks like you're hung up on my choice of words.
I suggest ignore that and instead just run the test-case.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25335
--- Comment #8 from amylaar at gcc dot gnu dot org 2006-01-31 16:32 ---
(In reply to comment #7)
> Re: "needing two rounds". Looks like you're hung up on my choice of words.
> I suggest ignore that and instead just run the test-case.
>
The patch for PR middle-end/24912 actually caused
--- Comment #9 from amylaar at gcc dot gnu dot org 2006-01-31 20:19 ---
(In reply to comment #0)
Visual inspection indicates that this patch fixes the testcase:
http://gcc.gnu.org/ml/gcc-patches/2006-01/msg02229.html
Can you confirm?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25
--- Comment #10 from hp at gcc dot gnu dot org 2006-01-31 22:10 ---
The bug is (unfortunately?) not exposed with trunk revision 110246.
Confirmed that the patch referred to in comment #9 fixes the bug
when applied to trunk revision 108225 (from around Dec 8).
Confirmed also that the p
--- Comment #11 from amylaar at gcc dot gnu dot org 2006-01-31 22:32
---
(In reply to comment #9)
P.S.: For the 4.1 branch and older release branches, it is sufficient to swap
the order of the calls to emit_reload_insns and subst_reloads.
--
http://gcc.gnu.org/bugzilla/show_bug.c
--- Comment #12 from amylaar at gcc dot gnu dot org 2006-02-10 17:42
---
I get the following code for rld-legit2.c with the patch I am currently
testing:
_g:
subq 4,$sp
move $srp,[$sp]
subq 40,$sp
movem $r8,[$sp]
move $r10,$srp
move.d $r11
--- Comment #13 from hp at gcc dot gnu dot org 2006-02-10 22:11 ---
(In reply to comment #12)
This code is exactly the same as the correct code I saw
for cris-axis-linux-gnu with the patch in comment #9, so I can confirm that
the emitted code is correct by visual inspection. On the othe
--- Comment #14 from amylaar at gcc dot gnu dot org 2006-02-13 15:33
---
Subject: Bug 25335
Author: amylaar
Date: Mon Feb 13 15:32:57 2006
New Revision: 110916
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=110916
Log:
Add PR marker for:
PR middle-end/25335
* re
--- Comment #15 from mmitchel at gcc dot gnu dot org 2006-02-14 09:17
---
OK for 4.1, should you wish to apply it there.
--
mmitchel at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #16 from joern dot rennecke at st dot com 2006-02-14 20:03
---
Subject: Re: [4.1/4.2 Regression] reload leaves insns from earlier passes
around: fatal for postinc
mmitchel at gcc dot gnu dot org wrote:
>--- Comment #15 from mmitchel at gcc dot gnu dot org 2006-02-14
--- Comment #17 from mark at codesourcery dot com 2006-02-15 01:56 ---
Subject: Re: [4.1/4.2 Regression] reload leaves insns
from earlier passes around: fatal for postinc
Joern RENNECKE wrote:
> I have started regression tests for i686-pc-linux-gnu native, X arm-elf,
> X cris-elf, X
--- Comment #18 from amylaar at gcc dot gnu dot org 2006-02-15 13:28
---
Subject: Bug 25335
Author: amylaar
Date: Wed Feb 15 13:28:05 2006
New Revision: 11
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=11
Log:
PR middle-end/25335
* reload1.c (gen_reload
--
mmitchel at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|4.1.0 |4.1.1
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25335
--- Comment #1 from pinskia at gcc dot gnu dot org 2005-12-10 04:00 ---
So this is a latent bug exposed by that fix which means this should be marked
as a regression.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #2 from hp at gcc dot gnu dot org 2005-12-10 04:53 ---
I mean for *this* test-case it's exposed by that fix. The leaving-behind
of reload-insns from earlier rounds for each new reload round is general.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25335
23 matches
Mail list logo
